https://github.com/HoBoIs updated
https://github.com/llvm/llvm-project/pull/83279
From 68200ecf3267d1b3940fa73c25c50ee706932a98 Mon Sep 17 00:00:00 2001
From: Botond Istvan Horvath
Date: Wed, 28 Feb 2024 13:09:15 +0100
Subject: [PATCH 1/2] Bugfix for choosing the more specialized overload
Ther
https://github.com/alejandro-alvarez-sonarsource updated
https://github.com/llvm/llvm-project/pull/83281
From a9419bc3ffa3d383a9373f0a116795162632dd40 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alejandro=20=C3=81lvarez=20Ayll=C3=B3n?=
Date: Wed, 28 Feb 2024 16:49:35 +0100
Subject: [PATCH] Reappl
https://github.com/alejandro-alvarez-sonarsource updated
https://github.com/llvm/llvm-project/pull/83281
From a9419bc3ffa3d383a9373f0a116795162632dd40 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alejandro=20=C3=81lvarez=20Ayll=C3=B3n?=
Date: Wed, 28 Feb 2024 16:49:35 +0100
Subject: [PATCH 1/2] Re
vinayakdsci wrote:
Done! I added the diagnostic to the TableGen file, and passed in only the
Record's source range instead of the whole expression's. Now only the lambda's
initializer is highlighted in the diagnostic. I hope this does it!
https://github.com/llvm/llvm-project/pull/83152
___
Botond =?utf-8?q?István_Horváth?Message-ID:
In-Reply-To:
https://github.com/erichkeane commented:
This needs tests plus a release note.
Also, there is a lot of repeated code between the pre-c++20 and c++20 versions
that will make this really hard to maintain, particularly with core issues,
Botond =?utf-8?q?István_Horváth?Message-ID:
In-Reply-To:
@@ -5548,13 +5514,113 @@ static bool isAtLeastAsSpecializedAs(Sema &S,
FunctionTemplateDecl *Sema::getMoreSpecializedTemplate(
FunctionTemplateDecl *FT1, FunctionTemplateDecl *FT2, SourceLocation Loc,
TemplatePa
Botond =?utf-8?q?István_Horváth?Message-ID:
In-Reply-To:
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/83279
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
Botond =?utf-8?q?István_Horváth?Message-ID:
In-Reply-To:
@@ -5548,13 +5514,113 @@ static bool isAtLeastAsSpecializedAs(Sema &S,
FunctionTemplateDecl *Sema::getMoreSpecializedTemplate(
FunctionTemplateDecl *FT1, FunctionTemplateDecl *FT2, SourceLocation Loc,
TemplatePa
Botond =?utf-8?q?István_Horváth?Message-ID:
In-Reply-To:
@@ -5333,11 +5333,31 @@ bool
Sema::CheckIfFunctionSpecializationIsImmediate(FunctionDecl *FD,
return false;
}
+static QualType GetImplicitObjectParameterTypeCXX20(ASTContext &Context,
+
Botond =?utf-8?q?István_Horváth?Message-ID:
In-Reply-To:
@@ -5333,11 +5333,31 @@ bool
Sema::CheckIfFunctionSpecializationIsImmediate(FunctionDecl *FD,
return false;
}
+static QualType GetImplicitObjectParameterTypeCXX20(ASTContext &Context,
erichkeane wro
asmok-g wrote:
Hello,
This patch makes the following line fail to compile
```
#elif ABSL_HAVE_BUILTIN(__builtin_cpu_supports)
if (__builtin_cpu_supports("avx2")) {
```
`error: invalid cpu feature string for builtin` when built on arm
https://github.com/llvm/llvm-project/pull/82378
_
https://github.com/jhuber6 created
https://github.com/llvm/llvm-project/pull/83282
Summary:
One recurring problem we have with the OpenMP libraries is that they are
potentially conflicting with ones found on the system, this occurs when
there are two copies and one is used for linking that it no
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Joseph Huber (jhuber6)
Changes
Summary:
One recurring problem we have with the OpenMP libraries is that they are
potentially conflicting with ones found on the system, this occurs when
there are two copies and one is used for linking
DanielKristofKiss wrote:
> Hello,
>
> This patch makes the second line fail to compile
>
> ```
> #elif ABSL_HAVE_BUILTIN(__builtin_cpu_supports)
> if (__builtin_cpu_supports("avx2")) {
> ```
>
> `error: invalid cpu feature string for builtin` when built on arm
expected to fail as __builtin_
https://github.com/labrinea edited
https://github.com/llvm/llvm-project/pull/81893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jhuber6 wrote:
So, I'm wondering if we could do a clang configuration file based solution for
this. The problem that I see now is that we'd like to make some clang
configuration files only active for a certain language. I think we already have
OS specific files and target specific files, so it
https://github.com/jeanPerier created
https://github.com/llvm/llvm-project/pull/83285
Polymorphic entity lowering status is good. The main remaining TODO is to allow
lowering of vector subscripted polymorphic entity, but this does not deserve
blocking all application using polymorphism.
Remov
@@ -983,7 +979,7 @@ void CodeGenPGO::mapRegionCounters(const Decl *D) {
// for most embedded applications. Setting a maximum value prevents the
// bitmap footprint from growing too large without the user's knowledge. In
// the future, this value could be adjusted with a c
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
@llvm/pr-subscribers-flang-fir-hlfir
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-flang-driver
Author: None (jeanPerier)
Changes
Polymorphic entity lowering status is good. The main remaining TODO is to allow
lowering of vector subscript
https://github.com/Jugst3r edited
https://github.com/llvm/llvm-project/pull/82448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2763,13 +2763,13 @@ void tools::addOpenMPDeviceRTL(const Driver &D,
const llvm::opt::ArgList &DriverArgs,
llvm::opt::ArgStringList &CC1Args,
StringRef BitcodeSuffix,
-
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/83282
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Jugst3r deleted
https://github.com/llvm/llvm-project/pull/82448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/balazske created
https://github.com/llvm/llvm-project/pull/83288
If a stream operation fails the position can become "indeterminate". This may
cause warning from the checker at a later operation. The new note tag shows the
place where the position becomes "indeterminate", th
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Balázs Kéri (balazske)
Changes
If a stream operation fails the position can become "indeterminate". This may
cause warning from the checker at a later operation. The new note tag shows the
place where the position becomes "indeterminate",
https://github.com/clementval approved this pull request.
Looks great! Thanks Jean to work on this!
https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/DanielKristofKiss updated
https://github.com/llvm/llvm-project/pull/82785
>From c5c2d720e822624fa7966297087b04e6b2fc2a86 Mon Sep 17 00:00:00 2001
From: Daniel Kiss
Date: Fri, 23 Feb 2024 17:12:26 +0100
Subject: [PATCH 1/2] [NFC][ARM][AArch64] Deduplicated code.
Add the SignR
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 9e1432069555d70e1f0148742e565b31d3ba8695
0d8b058dd8de3b29888f76554fca2d920410a4d7 --
@@ -1369,13 +1369,20 @@ class TargetInfo : public TransferrableTargetInfo,
}
struct BranchProtectionInfo {
-LangOptions::SignReturnAddressScopeKind SignReturnAddr =
-LangOptions::SignReturnAddressScopeKind::None;
-LangOptions::SignReturnAddressKeyKind SignK
https://github.com/tmatheson-arm approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/82785
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
balazske wrote:
Related to this, I want to change text "File position of the stream might be
'indeterminate'" to something better, or maybe only remove the `'` characters
from it.
https://github.com/llvm/llvm-project/pull/83288
___
cfe-commits mailin
https://github.com/DanielKristofKiss updated
https://github.com/llvm/llvm-project/pull/83277
>From 46f0334eb5c5edd15b657429f39f588cc7726072 Mon Sep 17 00:00:00 2001
From: Daniel Kiss
Date: Wed, 28 Feb 2024 15:18:31 +0100
Subject: [PATCH 1/2] Add branch protection attributes to the defaults.
Th
https://github.com/DanielKristofKiss updated
https://github.com/llvm/llvm-project/pull/82819
>From c5c2d720e822624fa7966297087b04e6b2fc2a86 Mon Sep 17 00:00:00 2001
From: Daniel Kiss
Date: Fri, 23 Feb 2024 17:12:26 +0100
Subject: [PATCH 1/4] [NFC][ARM][AArch64] Deduplicated code.
Add the SignR
ilovepi wrote:
Hmm, not sure why this comes out differently on windows in the presubmit.
https://github.com/llvm/llvm-project/pull/83159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/balazske updated
https://github.com/llvm/llvm-project/pull/83288
From f5aeb6e97e73d317d55189c656c31a84757f9636 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Bal=C3=A1zs=20K=C3=A9ri?=
Date: Wed, 28 Feb 2024 17:51:29 +0100
Subject: [PATCH] [clang][analyzer] Add StreamChecker note ta
balazske wrote:
fixed formatting errors
https://github.com/llvm/llvm-project/pull/83288
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ributzka approved this pull request.
https://github.com/llvm/llvm-project/pull/82552
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tmatheson-arm approved this pull request.
https://github.com/llvm/llvm-project/pull/83277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
balazske wrote:
Behavior of `fileno` is already changed in #81842.
I was thinking about that
> separate long-term solution
in last comment that it is already existing functionality (in StreamChecker and
other invalid pointer checkers).
Should we run again the checks (only modeling of `fileno`
@@ -16538,6 +16538,21 @@ void Sema::DiagnoseAlwaysNonNullPointer(Expr *E,
}
}
+ // Complain if we are converting a lambda expression to a boolean value
+ if (const auto *MCallExpr = dyn_cast(E)) {
+if (const auto *MRecordDecl = MCallExpr->getRecordDecl();
+
@@ -4127,8 +4127,8 @@ def ext_ms_impcast_fn_obj : ExtWarn<
"Microsoft extension">, InGroup;
def warn_impcast_pointer_to_bool : Warning<
-"address of%select{| function| array}0 '%1' will always evaluate to "
-"'true'">,
+"address of%select{| function| array| lambd
georgthegreat wrote:
@ldionne, gentle ping
https://github.com/llvm/llvm-project/pull/80443
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -628,9 +628,8 @@ void CGDebugInfo::CreateCompileUnit() {
// file was specified with an absolute path.
if (CSKind)
CSInfo.emplace(*CSKind, Checksum);
- llvm::DIFile *CUFile = DBuilder.createFile(
- remapDIPath(MainFileName), remapDIPath(getCurrentDirname()), CSI
@@ -16538,6 +16538,21 @@ void Sema::DiagnoseAlwaysNonNullPointer(Expr *E,
}
}
+ // Complain if we are converting a lambda expression to a boolean value
+ if (const auto *MCallExpr = dyn_cast(E)) {
+if (const auto *MRecordDecl = MCallExpr->getRecordDecl();
+
https://github.com/noxwell updated
https://github.com/llvm/llvm-project/pull/83175
>From 731463e1134b0821e518a3e8e521a2d44dd18a8e Mon Sep 17 00:00:00 2001
From: Aleksei Vetrov
Date: Tue, 27 Feb 2024 19:50:13 +
Subject: [PATCH] [Clang][DebugInfo] Use CGDebugInfo::createFile in
CGDebugInfo::
@@ -175,8 +192,19 @@ static void sanitizeDiagOpts(DiagnosticOptions &DiagOpts) {
DiagOpts.ShowCarets = false;
// Don't write out diagnostic file.
DiagOpts.DiagnosticSerializationFile.clear();
- // Don't emit warnings as errors (and all other warnings too).
- DiagOpts.Ig
pidgeon777 wrote:
@sam-mccall also pinging to assist @HighCommander4 🙂
https://github.com/llvm/llvm-project/pull/77556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Bäder
Date: 2024-02-28T18:14:12+01:00
New Revision: 95e036956f0a610027907df9a8f99d1f3c3b4cf5
URL:
https://github.com/llvm/llvm-project/commit/95e036956f0a610027907df9a8f99d1f3c3b4cf5
DIFF:
https://github.com/llvm/llvm-project/commit/95e036956f0a610027907df9a8f99d1f3c3b4cf5.diff
LO
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/83297
LZMA (Lempel-Ziv/Markov-chain Algorithm) provides better comparession rate than
zstd and zlib for clang-offload-bundler bundles which often contains large
number of similar entries.
This patch adds liblzma as
llvmbot wrote:
@llvm/pr-subscribers-llvm-support
@llvm/pr-subscribers-clang-driver
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
LZMA (Lempel-Ziv/Markov-chain Algorithm) provides better comparession rate than
zstd and zlib for clang-offload-bundler bundles which often contains large
number
jhuber6 wrote:
This seems to be adding an entirely new compression scheme to LLVM. I feel like
that should be a separate patch and the part where we make HIP use it is a
follow-up.
https://github.com/llvm/llvm-project/pull/83297
___
cfe-commits maili
noxwell wrote:
> Can you add a test for this?
I added a test for remapping of absolute path. It passes with this PR and fails
on trunk.
However, I found much bigger problem. This fix fails test
`debug-info-abspath.c`, more specifically the last case, which checks that CU
file has absolute pa
https://github.com/kiranchandramohan approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vinayakdsci deleted
https://github.com/llvm/llvm-project/pull/83152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Paul T Robinson
Date: 2024-02-28T12:36:28-05:00
New Revision: 15b6908afa9b74ae62821ff44179d659277eef69
URL:
https://github.com/llvm/llvm-project/commit/15b6908afa9b74ae62821ff44179d659277eef69
DIFF:
https://github.com/llvm/llvm-project/commit/15b6908afa9b74ae62821ff44179d659277eef69.dif
https://github.com/pogo59 closed https://github.com/llvm/llvm-project/pull/82422
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vinayakdsci updated
https://github.com/llvm/llvm-project/pull/83152
>From 640cbf94879217526e27884beefcbffa404fc082 Mon Sep 17 00:00:00 2001
From: Vinayak Dev
Date: Tue, 27 Feb 2024 18:05:29 +0530
Subject: [PATCH] [Clang][Sema]: Diagnose lambda to bool implicit casts
---
cla
vinayakdsci wrote:
@AaronBallman sorry for repeating the same mistake twice, I was unable to
correctly interpret what you wanted to say when you meant removing the lambda
body from the diagnostic.
I have made all the suggested changes and updated the tests. I hope I don't
come across as dumb :
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/83306
LZMA (Lempel-Ziv/Markov-chain Algorithm) provides better comparession
rate than zstd and zlib for clang-offload-bundler bundles which often
contains large number of similar entries.
This patch let clang-offload-
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
LZMA (Lempel-Ziv/Markov-chain Algorithm) provides better comparession
rate than zstd and zlib for clang-offload-bundler bundles which often
contains large number of similar entries.
This patch let clang-o
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/82552
>From ab971dc5d38808fc884d39112b68675cf9ce3868 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Wed, 21 Feb 2024 14:56:02 -0800
Subject: [PATCH] [InstallAPI] Hookup Input files & basic ASTVisitor
This patch
https://github.com/tblah approved this pull request.
LGTM. On a quick run this seems to pass everything I would expect it to in the
gfortran test suite. When you get time please could you enable the polymorphic
tests in the gfortran test suite and re-categorize the ones which now hit
different
yxsamliu wrote:
depends on https://github.com/llvm/llvm-project/pull/83297
https://github.com/llvm/llvm-project/pull/83306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Cyndy Ishida
Date: 2024-02-28T09:47:45-08:00
New Revision: c6cbf81c84b00ab0107a09f5b2d0183458a367b8
URL:
https://github.com/llvm/llvm-project/commit/c6cbf81c84b00ab0107a09f5b2d0183458a367b8
DIFF:
https://github.com/llvm/llvm-project/commit/c6cbf81c84b00ab0107a09f5b2d0183458a367b8.diff
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/82552
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/83306
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HoBoIs updated
https://github.com/llvm/llvm-project/pull/83279
From 68200ecf3267d1b3940fa73c25c50ee706932a98 Mon Sep 17 00:00:00 2001
From: Botond Istvan Horvath
Date: Wed, 28 Feb 2024 13:09:15 +0100
Subject: [PATCH 1/3] Bugfix for choosing the more specialized overload
Ther
Author: Alexandros Lamprineas
Date: 2024-02-28T17:49:59Z
New Revision: b42b7c8a123863d86db9abc8b6a1340b920f6573
URL:
https://github.com/llvm/llvm-project/commit/b42b7c8a123863d86db9abc8b6a1340b920f6573
DIFF:
https://github.com/llvm/llvm-project/commit/b42b7c8a123863d86db9abc8b6a1340b920f6573.di
https://github.com/labrinea closed
https://github.com/llvm/llvm-project/pull/81893
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tblah edited https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tblah wrote:
A few of the tests in the gfortran-testsuite using polymorphism fail at
runtime. I'm happy for this PR to be merged as it is and these bugs to be fixed
later.
https://github.com/llvm/llvm-project/pull/83285
___
cfe-commits mailing list
c
AaronBallman wrote:
> @AaronBallman sorry for repeating the same mistake twice, I was unable to
> correctly interpret what you wanted to say when you meant removing the lambda
> body from the diagnostic. I have made all the suggested changes and updated
> the tests. I hope I don't come across
https://github.com/AaronBallman approved this pull request.
LGTM! @Endilll, do you have any concerns?
https://github.com/llvm/llvm-project/pull/83152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinf
Author: Cyndy Ishida
Date: 2024-02-28T10:01:30-08:00
New Revision: 6101cf3164d77cfa4e2df75bf5262da9a2fb3d0a
URL:
https://github.com/llvm/llvm-project/commit/6101cf3164d77cfa4e2df75bf5262da9a2fb3d0a
DIFF:
https://github.com/llvm/llvm-project/commit/6101cf3164d77cfa4e2df75bf5262da9a2fb3d0a.diff
Sirraide wrote:
> Thank you for the fix! This seems to fix quite a few issues because it also
> addresses:
>
> #42411 #18121 #11039
I also checked for other open issues, but I see you were more successful than
me in finding some; I’ll take a look at those and add tests for them as well.
ht
jeanPerier wrote:
Thanks a lot for testing Tom!
I did not expect issues with gfortran tests. I will look into it before merging
and I made a simpler first patch thet will be less noisy to revert in case of
other issues that would break the build bots:
https://github.com/llvm/llvm-project/pull
https://github.com/Kalesh-Singh updated
https://github.com/llvm/llvm-project/pull/70251
>From 9a28c8670619cef36c24cfae426631098920fecc Mon Sep 17 00:00:00 2001
From: Kalesh Singh
Date: Wed, 25 Oct 2023 13:31:42 -0700
Subject: [PATCH 1/2] ANDROID: AArch64: Change default max-page-size from 4k to
@@ -10598,10 +10598,36 @@ QualType
Sema::DeduceTemplateSpecializationFromInitializer(
if (TemplateName.isDependent())
return SubstAutoTypeDependent(TSInfo->getType());
- // We can only perform deduction for class templates.
+ // We can only perform deduction for class
@@ -2258,6 +2258,94 @@ class ExtractTypeForDeductionGuide
}
};
+// Build a deduction guide with the specified parameter types.
+FunctionTemplateDecl *
+buildDeductionGuide(Sema &SemaRef, TemplateDecl *OriginalTemplate,
+TemplateParameterList *TemplatePara
@@ -10598,10 +10598,36 @@ QualType
Sema::DeduceTemplateSpecializationFromInitializer(
if (TemplateName.isDependent())
return SubstAutoTypeDependent(TSInfo->getType());
- // We can only perform deduction for class templates.
+ // We can only perform deduction for class
alexfh wrote:
This makes the clang diverge from the GCC behavior, for example, GCC happily
compiles `(void)__builtin_cpu_supports("neon");` on x86, but clang doesn't
after this patch: https://gcc.godbolt.org/z/a367Pd7sE
https://github.com/llvm/llvm-project/pull/82378
__
@@ -2258,6 +2258,94 @@ class ExtractTypeForDeductionGuide
}
};
+// Build a deduction guide with the specified parameter types.
+FunctionTemplateDecl *
+buildDeductionGuide(Sema &SemaRef, TemplateDecl *OriginalTemplate,
+TemplateParameterList *TemplatePara
usx95 wrote:
I tried using the EHCleanups as the only source of cleanups for these
branch-in-expr. https://github.com/llvm/llvm-project/pull/83224 is an attempt.
I do not like it very much.
The first challenge is to add these cleanups to the stack even when exceptions
are disabled. Some of th
mikerice1969 wrote:
Hi @josemonsalve2 if you want to help move this along it would be best to break
this into at least three separate PRs.
1) Parsing/Sema/Serialization
2) CodeGen
3) Extension checking framework
In whatever order makes sense to you.
We've added several extension directives in o
Sirraide wrote:
> #11039
It does not fix this one because whereas GCC supports this, we don’t:
```c++
struct foo {
void f();
};
void foo::f() __restrict {
}
```
because `__restrict` here is treated like `const`, so it has to be present on
both the declaration and definition; however, GCC
vinayakdsci wrote:
@AaronBallman does this PR look good to go, or should I make any changes and
updates? I would really be grateful for any pointers on how this can be
improved.
Thanks a lot!
https://github.com/llvm/llvm-project/pull/81127
___
cfe-c
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/83187
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SunilKuravinakop updated
https://github.com/llvm/llvm-project/pull/82604
>From 6444ef3f60a9f8ed8b3f413997259fe5006396b7 Mon Sep 17 00:00:00 2001
From: Sunil Kuravinakop
Date: Tue, 20 Feb 2024 11:29:49 -0600
Subject: [PATCH 1/6] Accept multiple init clauses in interop directiv
SunilKuravinakop wrote:
The recent commit also takes care of the feedback provided by Alexey.
https://github.com/llvm/llvm-project/pull/82604
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
https://github.com/Sirraide updated
https://github.com/llvm/llvm-project/pull/83187
>From e9df22166fd5768e11273446cb9e52953e55cd5d Mon Sep 17 00:00:00 2001
From: Sirraide
Date: Tue, 27 Feb 2024 22:07:23 +0100
Subject: [PATCH 1/4] [Clang] [Sema] Handle __restrict-qualified member
functions prop
Sirraide wrote:
> however, GCC accepts this; should we do so as well?
If so, should that be a separate pr, perhaps?
https://github.com/llvm/llvm-project/pull/83187
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/
https://github.com/ilovepi updated
https://github.com/llvm/llvm-project/pull/83159
>From 0a37b8e52fe5e17fff3f6e88bf25f1969bbe8bb4 Mon Sep 17 00:00:00 2001
From: Paul Kirth
Date: Tue, 27 Feb 2024 17:43:58 +
Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF-8
ilovepi wrote:
Seems like "PIE Level" is different, Lets see if passing `-fPIE` will satisfy
the windows bot
https://github.com/llvm/llvm-project/pull/83159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
@@ -7023,31 +7023,47 @@ void CodeGenFunction::EmitOMPInteropDirective(const
OMPInteropDirective &S) {
S.getSingleClause())) &&
"OMPNowaitClause clause is used separately in OMPInteropDirective.");
- if (const auto *C = S.getSingl
Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
Message-ID:
In-Reply-To:
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/83279
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-
Botond =?utf-8?q?Istv=C3=A1n_Horv=C3=A1th?=,Botond Istvan Horvath
Message-ID:
In-Reply-To:
https://github.com/erichkeane commented:
This is close I think, just needs tests + ReleaseNotes.rst entry.
https://github.com/llvm/llvm-project/pull/83279
__
Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
Message-ID:
In-Reply-To:
@@ -5548,13 +5504,100 @@ static bool isAtLeastAsSpecializedAs(Sema &S,
FunctionTemplateDecl *Sema::getMoreSpecializedTemplate(
FunctionTemplateDecl *FT1, FunctionTemplateDecl *FT2, SourceLoc
Botond =?utf-8?q?István_Horváth?=,Botond Istvan Horvath
Message-ID:
In-Reply-To:
@@ -5548,13 +5504,100 @@ static bool isAtLeastAsSpecializedAs(Sema &S,
FunctionTemplateDecl *Sema::getMoreSpecializedTemplate(
FunctionTemplateDecl *FT1, FunctionTemplateDecl *FT2, SourceLoc
https://github.com/overmighty created
https://github.com/llvm/llvm-project/pull/83313
Our implementation previously accepted signed arguments and performed
integer promotion on the argument. GCC's implementation requires an
unsigned argument and does not perform integer promotion on it.
See htt
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: OverMighty (overmighty)
Changes
Our implementation previously accepted signed arguments and performed
integer promotion on the argument. GCC's implementation requires an
unsigned argument and does not perform integer promotion on it.
See h
101 - 200 of 345 matches
Mail list logo
