@@ -285,15 +285,43 @@ void ExprEngine::VisitCast(const CastExpr *CastE, const
Expr *Ex,
ExplodedNodeSet dstPreStmt;
getCheckerManager().runCheckersForPreStmt(dstPreStmt, Pred, CastE, *this);
- if (CastE->getCastKind() == CK_LValueToRValue ||
- CastE->getCastKind() =
Author: erichkeane
Date: 2025-05-09T13:08:54-07:00
New Revision: 73ca11625f40d4bccb9b7219c163fb8b543c4da4
URL:
https://github.com/llvm/llvm-project/commit/73ca11625f40d4bccb9b7219c163fb8b543c4da4
DIFF:
https://github.com/llvm/llvm-project/commit/73ca11625f40d4bccb9b7219c163fb8b543c4da4.diff
LO
Author: Oliver Hunt
Date: 2025-05-09T13:12:09-07:00
New Revision: 65a6cbde5bb074ce377cb1aa6145241ad1719c17
URL:
https://github.com/llvm/llvm-project/commit/65a6cbde5bb074ce377cb1aa6145241ad1719c17
DIFF:
https://github.com/llvm/llvm-project/commit/65a6cbde5bb074ce377cb1aa6145241ad1719c17.diff
L
https://github.com/ojhunt closed
https://github.com/llvm/llvm-project/pull/137580
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/138055
>From e1b21cc4d33dab2e100346443c998a678afc3dab Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 30 Apr 2025 23:14:07 +
Subject: [PATCH 1/6] pre-req: define missing lexer tokens for flags
---
.../clan
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/138007
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-s
https://github.com/bcardosolopes approved this pull request.
https://github.com/llvm/llvm-project/pull/139134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,178 @@
+// Test that modifications to a common header (imported from both a PCH and a
TU)
+// cause rebuilds of dependent modules imported from the TU on incremental
build.
+
+// RUN: rm -rf %t
+// RUN: split-file %s %t
+
+//--- module.modulemap
+module mod_common { h
https://github.com/bcardosolopes commented:
Thanks for following up on this!
Once it lands we should probably backport to incubator too.
https://github.com/llvm/llvm-project/pull/139304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
@@ -1395,6 +1395,26 @@ LogicalResult cir::VecCreateOp::verify() {
return success();
}
+//===--===//
+// VecExtractOp
+//===--===//
+
+OpFol
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/139277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: erichkeane
Date: 2025-05-09T12:54:42-07:00
New Revision: 31fd77aa51a643245f8eb277483554509b771832
URL:
https://github.com/llvm/llvm-project/commit/31fd77aa51a643245f8eb277483554509b771832
DIFF:
https://github.com/llvm/llvm-project/commit/31fd77aa51a643245f8eb277483554509b771832.diff
LO
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/137690
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida updated
https://github.com/llvm/llvm-project/pull/139318
>From 9a733c05d8ab18c1a7c45fc159718994a544f722 Mon Sep 17 00:00:00 2001
From: Cyndy Ishida
Date: Fri, 9 May 2025 12:16:14 -0700
Subject: [PATCH 1/2] [llvm][Support] Avoid setting minor/subminor/build in
Ver
Artem-B wrote:
@cgmb
> I would suggest that we should either (a) change the default GPU target to
> native and make the failure to detect the user’s GPU into a hard compiler
> error, or (b) change the default GPU target to SPIR-V so that it works on
> every machine.
The thing is that the se
https://github.com/ziqingluo-90 updated
https://github.com/llvm/llvm-project/pull/139188
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,s
@@ -5790,7 +5805,11 @@ class OMPReverseDirective final : public
OMPLoopTransformationDirective {
explicit OMPReverseDirective(SourceLocation StartLoc, SourceLocation EndLoc)
: OMPLoopTransformationDirective(OMPReverseDirectiveClass,
ziqingluo-90 wrote:
Thanks for reviewing!
https://github.com/llvm/llvm-project/pull/139188
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AmrDeveloper wrote:
> Thank you for the fix! I've added a few more reviewers. This is a pretty
> general problem, so I think we may want additional test coverage for
> basically any of the OpenMP clauses which accept an integer argument. For
> example, this is another related issue: #139268 --
https://github.com/ojhunt edited
https://github.com/llvm/llvm-project/pull/136828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/139246
When forming an invalid function type, we were not diagnosing it if the call
was dependent.
However, we later rely on the function type to be sensible during argument
deduction.
We now diagnose anything that
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/139118
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/138864
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-se
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 HEAD~1 HEAD --extensions cpp,h --
clang/test/CodeGenCXX/Inputs/vtable-debug-info-inhe
Author: Martin Storsjö
Date: 2025-05-09T16:04:57+03:00
New Revision: 9c67ae861ccb0312fc680414a8441892d169d14a
URL:
https://github.com/llvm/llvm-project/commit/9c67ae861ccb0312fc680414a8441892d169d14a
DIFF:
https://github.com/llvm/llvm-project/commit/9c67ae861ccb0312fc680414a8441892d169d14a.diff
@@ -1538,11 +1533,32 @@ def CUDAShared : InheritableAttr {
}
def : MutualExclusions<[CUDAConstant, CUDAShared, HIPManaged]>;
-def SYCLKernel : InheritableAttr {
- let Spellings = [Clang<"sycl_kernel">];
- let Subjects = SubjectList<[FunctionTmpl]>;
- let LangOpts = [SYCLDev
https://github.com/cyndyishida approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/138920
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/139057
>From 24e912a3ded13660ff094d488b8529d63ff0d821 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Thu, 8 May 2025 11:43:06 +0200
Subject: [PATCH 1/3] [Clang] Fix handling of pack indexing types in
constraints
nightlark wrote:
Yes, still looking into this -- got a bit busier through around mid-May though.
https://github.com/llvm/llvm-project/pull/125806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/139277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -9659,6 +9659,42 @@ void Sema::CheckMemaccessArguments(const CallExpr *Call,
if (BId == Builtin::BIbzero && !FirstArgTy->getAs())
return;
+ // Try to detect a relocation operation
+ if (getLangOpts().CPlusPlus &&
+ (BId == Builtin::BImemmove || BId == Builtin::
@@ -1538,11 +1533,32 @@ def CUDAShared : InheritableAttr {
}
def : MutualExclusions<[CUDAConstant, CUDAShared, HIPManaged]>;
-def SYCLKernel : InheritableAttr {
- let Spellings = [Clang<"sycl_kernel">];
- let Subjects = SubjectList<[FunctionTmpl]>;
- let LangOpts = [SYCLDev
@@ -4549,6 +4549,10 @@ class PackIndexingExpr final
bool isFullySubstituted() const { return FullySubstituted; }
+ bool isPartiallySubstituted() const {
cor3ntin wrote:
I got rid of it entierly, it's cleaner
https://github.com/llvm/llvm-project/pull/139
@@ -0,0 +1,93 @@
+//RUN: %libomp-cxx-compile -fopenmp-version=60 && %libomp-run
+#include
+#include
+#include "omp_testsuite.h"
+
+#define N 10
+class Sum {
+ int val;
+
+public:
+ Sum(int v = 0) : val(v) {}
+ Sum operator+(const Sum &rhs) const { return Sum(val + rhs.val);
@@ -33,7 +33,7 @@ T f1(T t1, U u1, int i1, T** tpp)
i1 = t1[u1];
i1 *= t1;
- i1(u1, t1); // expected-error {{called object type 'int' is not a function
or function pointer}}
+ i1(u1, t1);
cor3ntin wrote:
It is because that function is not instantiated
https://github.com/YutongZhuu updated
https://github.com/llvm/llvm-project/pull/133653
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Aaron Ballman (AaronBallman)
Changes
You cannot get the integer constant expression's value if the expression
contains errors.
Fixes https://github.com/llvm/llvm-project/issues/139267
---
Full diff: https://github.com/llvm/llvm-project/p
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/139276
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-
cor3ntin wrote:
> Also, should this come w/ a release note? Or does this just count as part of
> adding support for trivial relocation?
Yup, this is "implement trivial relocation". There are more PRs coming!
https://github.com/llvm/llvm-project/pull/139104
___
Author: Ties Stuij
Date: 2025-05-09T16:24:48+01:00
New Revision: 269f5fe91ef9abec5dcdc574b5fb7e7f35269f29
URL:
https://github.com/llvm/llvm-project/commit/269f5fe91ef9abec5dcdc574b5fb7e7f35269f29
DIFF:
https://github.com/llvm/llvm-project/commit/269f5fe91ef9abec5dcdc574b5fb7e7f35269f29.diff
LO
Author: Aaron Ballman
Date: 2025-05-09T11:25:41-04:00
New Revision: 51ca3cbb2b91a0361620248b6e3e22da1a68eb69
URL:
https://github.com/llvm/llvm-project/commit/51ca3cbb2b91a0361620248b6e3e22da1a68eb69
DIFF:
https://github.com/llvm/llvm-project/commit/51ca3cbb2b91a0361620248b6e3e22da1a68eb69.diff
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/139276
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`llvm-clang-x86_64-sie-ubuntu-fast` running on `sie-linux-worker` while
building `clang` at step 6 "test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/144/builds/24756
https://github.com/Fznamznon edited
https://github.com/llvm/llvm-project/pull/139277
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jansvoboda11 closed
https://github.com/llvm/llvm-project/pull/139091
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jansvoboda11 closed
https://github.com/llvm/llvm-project/pull/133827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman created
https://github.com/llvm/llvm-project/pull/139280
You cannot get the integer constant expression's value if the expression
contains errors.
Fixes https://github.com/llvm/llvm-project/issues/139267
Rate limit · GitHub
@@ -105,3 +105,11 @@ int main(int argc, char **argv) {
return (tmain(argc) + tmain(argv[0][0])); // expected-note
{{in instantiation of function template specialization 'tmain'
requested here}} expected-note {{in instantiation of function template
specialization 'tmain' re
llvmbot wrote:
@llvm/pr-subscribers-flang-openmp
@llvm/pr-subscribers-openmp
Author: Johannes Doerfert (jdoerfert)
Changes
The device and implementation set should trigger elision of tokens if they do
not match statically in a begin/end declare variant. This simply extends the
logic from
@@ -1,3 +1,3 @@
// RUN: %clang -### -nogpulib -nogpuinc -c %s 2>&1 | FileCheck %s
-// CHECK: {{.*}}clang{{.*}}"-target-cpu" "gfx906"
+// CHECK: {{.*}}clang{{.*}}"-triple" "{{amdgcn|spirv64}}-amd-amdhsa"{{.*}}
"-target-cpu" "{{amdgcnspirv|gfx.*}}"
arsenm wrote:
https://github.com/PiJoules closed
https://github.com/llvm/llvm-project/pull/139172
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: PiJoules
Date: 2025-05-09T10:19:52-07:00
New Revision: a6385a87a2e5537f0790494ebe8bb4c3cc9506b9
URL:
https://github.com/llvm/llvm-project/commit/a6385a87a2e5537f0790494ebe8bb4c3cc9506b9
DIFF:
https://github.com/llvm/llvm-project/commit/a6385a87a2e5537f0790494ebe8bb4c3cc9506b9.diff
LOG:
Author: erichkeane
Date: 2025-05-09T09:49:41-07:00
New Revision: c8539f7269d3ecb1246b7081378f694938ebbbf4
URL:
https://github.com/llvm/llvm-project/commit/c8539f7269d3ecb1246b7081378f694938ebbbf4
DIFF:
https://github.com/llvm/llvm-project/commit/c8539f7269d3ecb1246b7081378f694938ebbbf4.diff
LO
https://github.com/sarnex edited
https://github.com/llvm/llvm-project/pull/137882
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -60,3 +61,98 @@ void f3() {
f2(0);
f2(0); // expected-error {{no matching function for call to 'f2'}}
}
+
+#if __cplusplus >= 202002L
+namespace GH138657 {
+template // #gh138657-template-head
+class meta {};
+template
+class meta {}; // expected-error {{called object t
JungPhilipp wrote:
> Your branch seems to be way behind current `main`, and
> `clang-tools-extra/docs/ReleaseNotes.rst` seems to be from already released
> 20th version of LLVM. (I don't know how can it even show some kind of diff)
> Could you please rebase on fresh `main` branch.
Done
https
https://github.com/YutongZhuu reopened
https://github.com/llvm/llvm-project/pull/133653
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jdoerfert created
https://github.com/llvm/llvm-project/pull/139287
The device and implementation set should trigger elision of tokens if they do
not match statically in a begin/end declare variant. This simply extends the
logic from the device set only and includes the imple
Author: Jan Svoboda
Date: 2025-05-09T10:32:48-07:00
New Revision: 94ae5f9e877be7490687c35ec9883ff9746721d6
URL:
https://github.com/llvm/llvm-project/commit/94ae5f9e877be7490687c35ec9883ff9746721d6
DIFF:
https://github.com/llvm/llvm-project/commit/94ae5f9e877be7490687c35ec9883ff9746721d6.diff
L
https://github.com/jansvoboda11 closed
https://github.com/llvm/llvm-project/pull/138920
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jeremyd2019 wrote:
I think next on this would be to hook up so that it can call ld directly.
https://github.com/llvm/llvm-project/pull/135691
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co
https://github.com/eZWALT edited
https://github.com/llvm/llvm-project/pull/139293
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
@Sirraide, wait, did we all miss this PR? I am so sorry
https://github.com/llvm/llvm-project/pull/83855
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/alexey-bataev approved this pull request.
https://github.com/llvm/llvm-project/pull/139287
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Artem-B wrote:
@jhuber6 do you think can we use `native` instead? I think it would be a
somewhat better option here.
If we have to choose a GPU variant by default, we may as well choose the actual
GPU, rather than a conditional choice between generic SPIR-V or an old GPU,
which has the disadva
@@ -530,6 +530,12 @@ OpenMP Support
- Added support 'no_openmp_constructs' assumption clause.
- Added support for 'self_maps' in map and requirement clause.
- Added support for 'omp stripe' directive.
+- Fixed a crashing bug with ``omp unroll partial`` if the argument to
+ ``p
jhuber6 wrote:
> @jhuber6 do you think can we use `native` instead? I think it would be a
> somewhat better option here. If we have to choose a GPU variant by default,
> we may as well choose the actual GPU, rather than a conditional choice
> between generic SPIR-V or an old GPU, which has the
Author: Daniel Paoliello
Date: 2025-05-09T10:42:10-07:00
New Revision: 72c3ed67457e8f6bb0c953ae57e1a6d2ab6e4410
URL:
https://github.com/llvm/llvm-project/commit/72c3ed67457e8f6bb0c953ae57e1a6d2ab6e4410
DIFF:
https://github.com/llvm/llvm-project/commit/72c3ed67457e8f6bb0c953ae57e1a6d2ab6e4410.di
https://github.com/lamb-j approved this pull request.
https://github.com/llvm/llvm-project/pull/137623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: erichkeane
Date: 2025-05-09T10:52:23-07:00
New Revision: 24038650d9ca5d66b07d3075afdebe81012ab1f2
URL:
https://github.com/llvm/llvm-project/commit/24038650d9ca5d66b07d3075afdebe81012ab1f2
DIFF:
https://github.com/llvm/llvm-project/commit/24038650d9ca5d66b07d3075afdebe81012ab1f2.diff
LO
@@ -530,6 +530,12 @@ OpenMP Support
- Added support 'no_openmp_constructs' assumption clause.
- Added support for 'self_maps' in map and requirement clause.
- Added support for 'omp stripe' directive.
+- Fixed a crashing bug with ``omp unroll partial`` if the argument to
+ ``p
Juan Manuel Martinez =?utf-8?q?Caamaño?=
Message-ID:
In-Reply-To:
lamb-j wrote:
> module maps don't get preprocessed, so I don't think it matters what that is
> set to. For normal module.modulemap files `Preprocessed` is `false`. I think
> it's fine to remove the true and see if anything bre
https://github.com/dpaoliello closed
https://github.com/llvm/llvm-project/pull/129142
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/139246
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-
Author: cor3ntin
Date: 2025-05-09T19:59:20+02:00
New Revision: 52b18b4e82d412a7d755e89591c6ebcc41c257a1
URL:
https://github.com/llvm/llvm-project/commit/52b18b4e82d412a7d755e89591c6ebcc41c257a1
DIFF:
https://github.com/llvm/llvm-project/commit/52b18b4e82d412a7d755e89591c6ebcc41c257a1.diff
LOG:
cor3ntin wrote:
Sorry about that, folks. And thanks for the minimal repro, it's greatly
appreciated.
I relanded the PR here https://github.com/llvm/llvm-project/pull/139246
I hope that resolves the issue you found
https://github.com/llvm/llvm-project/pull/138731
@@ -60,3 +61,98 @@ void f3() {
f2(0);
f2(0); // expected-error {{no matching function for call to 'f2'}}
}
+
+#if __cplusplus >= 202002L
+namespace GH138657 {
+template // #gh138657-template-head
+class meta {};
+template
+class meta {}; // expected-error {{called object t
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/139246
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DenisGZM updated
https://github.com/llvm/llvm-project/pull/133107
>From c46eda67cd7434dcce5c1f29125a940dc4ff64ba Mon Sep 17 00:00:00 2001
From: Denis Gerasimov
Date: Wed, 26 Mar 2025 18:29:38 +0300
Subject: [PATCH 1/8] [CLANG] Enable alignas after GNU attributes
---
clang/l
https://github.com/Lancern approved this pull request.
https://github.com/llvm/llvm-project/pull/139159
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cor3ntin (cor3ntin)
Changes
When forming an invalid function type, we were not diagnosing it if the call
was dependent.
However, we later rely on the function type to be sensible during argument
deduction.
We now diagnose anything that
@@ -1476,8 +1478,8 @@ namespace {
}
}
-TemplateArgument
-getTemplateArgumentPackPatternForRewrite(const TemplateArgument &TA) {
+TemplateArgument getTemplateArgumentorUnsubstitutedExpansionPattern(
zyn0217 wrote:
```suggestion
Templa
https://github.com/zyn0217 approved this pull request.
one nit, else LG!
https://github.com/llvm/llvm-project/pull/139057
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Prabhuk closed
https://github.com/llvm/llvm-project/pull/139179
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
asmok-g wrote:
@AaronBallman I met another case, giving the same error even after the fix.
Sorry I only noticed it now:
```
const char *p;
const char *pend;
...
return (free (compile_stack.stack), p == pend ? REG_EBRACE : REG_BADBR));
```
Both REG_EBRACE, REG_BADBR are `reg_errcode_t `
https:/
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 HEAD~1 HEAD --extensions h,cpp --
clang/test/OpenMP/for_private_reduction_codegen.cpp
https://github.com/arsenm updated
https://github.com/llvm/llvm-project/pull/138866
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-se
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`amdgpu-offload-rhel-8-cmake-build-only` running on `rocm-docker-rhel-8` while
building `flang` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/204/builds/8855
Here is the releva
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/139276
When profiling a pack indexing that has been partially substituted, we should
profile the expansions, rather than the pattern itseld
This is a better approach to #139057
This mirrors the fix done for SizeOfPa
llvmbot wrote:
@llvm/pr-subscribers-clang-modules
Author: cor3ntin (cor3ntin)
Changes
When profiling a pack indexing that has been partially substituted, we should
profile the expansions, rather than the pattern itseld
This is a better approach to #139057
This mirrors the fix done for S
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 HEAD~1 HEAD --extensions h,cpp --
clang/include/clang/AST/ASTContext.h clang/include/
https://github.com/zyn0217 approved this pull request.
Thanks for the iteration!
https://github.com/llvm/llvm-project/pull/139276
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4549,6 +4549,10 @@ class PackIndexingExpr final
bool isFullySubstituted() const { return FullySubstituted; }
+ bool isPartiallySubstituted() const {
+return isValueDependent() && TransformedExpressions;
+ };
zyn0217 wrote:
Trailing semicolon
htt
@@ -4549,6 +4549,10 @@ class PackIndexingExpr final
bool isFullySubstituted() const { return FullySubstituted; }
+ bool isPartiallySubstituted() const {
zyn0217 wrote:
Please document the difference with FullySubstituted
https://github.com/llvm/llvm-proj
https://github.com/YutongZhuu edited
https://github.com/llvm/llvm-project/pull/133653
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
After chatting with @zyn0217, we realized there was a better approach. #139276
https://github.com/llvm/llvm-project/pull/139057
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Aaron Ballman (AaronBallman)
Changes
We were failing to pass a required argument when emitting the diagnostic, so
the source range was being used in place of an index. This caused a failed
assertion due to the incorrect index.
Fixes #139
https://github.com/AaronBallman created
https://github.com/llvm/llvm-project/pull/139277
We were failing to pass a required argument when emitting the diagnostic, so
the source range was being used in place of an index. This caused a failed
assertion due to the incorrect index.
Fixes #139266
https://github.com/erichkeane approved this pull request.
I like this better too.
https://github.com/llvm/llvm-project/pull/139276
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/thurstond updated
https://github.com/llvm/llvm-project/pull/139149
>From c2a5ecf9d89b4942fb1e95c7a7713585155099d0 Mon Sep 17 00:00:00 2001
From: Thurston Dang
Date: Thu, 8 May 2025 20:18:22 +
Subject: [PATCH 1/4] [cfi][NFCI] Pre-commit -fsanitize-annotate-debug-info
test
101 - 200 of 553 matches
Mail list logo
