https://github.com/jmorse approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/134638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/141092
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/141245
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nico wrote:
Reading files from unit tests is a bit of an anti-pattern: It requires putting
the absolute path to the tests somewhere (you're doing this with a config file
here), and it means you easily run tests on a different machine than the one
you're building on. Could you restructure this
@@ -5753,7 +5753,7 @@ void CodeGenFunction::EmitOMPScanDirective(const
OMPScanDirective &S) {
CGM.getOpenMPRuntime().emitReduction(
*this, ParentDir.getEndLoc(), Privates, LHSs, RHSs, ReductionOps,
{/*WithNowait=*/true, /*SimpleReduction=*/true,
-
https://github.com/alexey-bataev approved this pull request.
LG with nits
https://github.com/llvm/llvm-project/pull/134709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/140874
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/alexey-bataev edited
https://github.com/llvm/llvm-project/pull/134709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3947,7 +3947,7 @@ static void emitScanBasedDirective(
CGF.CGM.getOpenMPRuntime().emitReduction(
CGF, S.getEndLoc(), Privates, LHSs, RHSs, ReductionOps,
{/*WithNowait=*/true, /*SimpleReduction=*/true,
- /*IsPrivateVarReduction*/{}, OMPD_un
eZWALT wrote:
> > > What I see in the source code that it is used as a boolean flag. Can we
> > > transform it to bool? There is no need to keep it integer
> >
> >
> > Please could you cite the exact line? I'm not sure if you are refering to
> > the logic inside checkTransformableLoopNest or
higher-performance wrote:
@efriedma-quic Ah I figured out how to test this - just added a test.
https://github.com/llvm/llvm-project/pull/141133
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe
https://github.com/higher-performance updated
https://github.com/llvm/llvm-project/pull/141133
>From 00e0cf90f1d53ec8b2472bd740066016ee88f584 Mon Sep 17 00:00:00 2001
From: higher-performance
Date: Thu, 22 May 2025 16:30:29 -0400
Subject: [PATCH] Include [[clang::require_explicit_initialization
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`llvm-clang-x86_64-gcc-ubuntu` running on `sie-linux-worker3` while building
`clang,llvm` at step 6 "test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/174/builds/18255
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/14
>From 2a56a5d8db006bd466db00989806d548ce042b37 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Thu, 22 May 2025 19:38:08 +0200
Subject: [PATCH 1/3] [CIR] Allow use different Int types together in Vec Shif
@@ -1427,13 +1427,13 @@ OpFoldResult cir::SelectOp::fold(FoldAdaptor adaptor) {
//===--===//
LogicalResult cir::ShiftOp::verify() {
mlir::Operation *op = getOperation();
- mlir::Type resType = getResult().g
@@ -1917,3 +1917,190 @@ ExprResult Sema::BuildExpressionTrait(ExpressionTrait
ET, SourceLocation KWLoc,
return new (Context)
ExpressionTraitExpr(KWLoc, ET, Queried, Value, RParen, Context.BoolTy);
}
+
+static std::optional StdNameToTypeTrait(StringRef Name) {
+ return
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `openmp-s390x-linux`
running on `systemz-1` while building `clang,llvm` at step 6 "test-openmp".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/88/builds/11980
Here is the relevant piece of the build
sarnex wrote:
Ah ok thank you! I'm surprised this work effects anything as I was mostly doing
it for upstream OMP SPV which doesnt' work yet :)
https://github.com/llvm/llvm-project/pull/137187
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
https://github.com/ilovepi created
https://github.com/llvm/llvm-project/pull/141269
In #138062 it was brought up that this was an anti-pattern. We'll need
to Migrate all of the mustache unittests to lit tests, and disable them
until tool support lands.
>From a675e0827d8bee1cfff9e095bc7eb394f520
https://github.com/ilovepi ready_for_review
https://github.com/llvm/llvm-project/pull/141269
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/141041
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
qinkunbao wrote:
https://github.com/llvm/llvm-project/pull/141270 can solve the issue but I
don't know what is the reason.
https://github.com/llvm/llvm-project/pull/140964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/
eZWALT wrote:
@alexey-bataev not sure what happened before with this build system, but now
everything works as expected. Thanks for the fast replies and have a nice
weekend!
https://github.com/llvm/llvm-project/pull/139293
___
cfe-commits mailing l
https://github.com/DanielCChen approved this pull request.
LGTM. Thanks.
https://github.com/llvm/llvm-project/pull/140850
___
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-tools-extra
Author: Paul Kirth (ilovepi)
Changes
In #138062 it was brought up that this was an anti-pattern. We'll need
to Migrate all of the mustache unittests to lit tests, and disable them
until tool support lands.
---
Full diff: https://github
https://github.com/ilovepi edited
https://github.com/llvm/llvm-project/pull/141269
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ilovepi edited
https://github.com/llvm/llvm-project/pull/141269
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/140964
>From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Wed, 21 May 2025 22:12:13 +
Subject: [PATCH 1/5] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UT
nico wrote:
> As an alternative, we could disable the unit tests for the new feature, since
> its expected to be experimental for some time, and move the testing into lit
> tests.
That sounds like a good way forward to me :)
https://github.com/llvm/llvm-project/pull/138062
___
ilovepi wrote:
* **#141269** https://app.graphite.dev/github/pr/llvm/llvm-project/141269?utm_source=stack-comment-icon";
target="_blank">https://static.graphite.dev/graphite-32x32-black.png"; alt="Graphite"
width="10px" height="10px"/> 👈 https://app.graphite.dev/github/pr/llvm/llvm-project/1412
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
placement-new'ing an object with a dead base object is not allowed, so we need
to check all the pointer bases.
---
Full diff: https://github.com/llvm/llvm-project/pull/141272.diff
7 Files Affected:
- (mod
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/141272
placement-new'ing an object with a dead base object is not allowed, so we need
to check all the pointer bases.
>From b926faa034774260871133e1eaa415f4b0bb8721 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C
qinkunbao wrote:
This PR change converts Matcher::Globs from StringMap to vector. However, after
the change, some unit tests are broken.
For example,
```
// build/unittests/Support/./SupportTests
--gtest_filter=SpecialCaseListTest.Basic
TEST_F(SpecialCaseListTest, Basic) {
std::unique_ptr SC
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/140964
>From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Wed, 21 May 2025 22:12:13 +
Subject: [PATCH 1/5] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UT
https://github.com/inbelic commented:
How does this/will this handle enforcing that it is a read-only buffer?
https://github.com/llvm/llvm-project/pull/141086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/petrhosek approved this pull request.
https://github.com/llvm/llvm-project/pull/141269
___
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: Ryosuke Niwa (rniwa)
Changes
This PR adds the WebKit checker support for
[[clang::annotate_type("webkit.pointerconversion")]].
When this attribute is set on the return value of a function, the function is
treated as safe to call anywhere
Author: Paul Kirth
Date: 2025-05-23T11:47:20-07:00
New Revision: 524ef164a745a364f80c11c9a8ddb58321ed408f
URL:
https://github.com/llvm/llvm-project/commit/524ef164a745a364f80c11c9a8ddb58321ed408f
DIFF:
https://github.com/llvm/llvm-project/commit/524ef164a745a364f80c11c9a8ddb58321ed408f.diff
LO
https://github.com/ilovepi closed
https://github.com/llvm/llvm-project/pull/141269
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 approved this pull request.
https://github.com/llvm/llvm-project/pull/141269
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ilovepi wrote:
#141269 should fix the issue for now. I'll clean up the rest of the stack to
avoid reintroducing this problem.
https://github.com/llvm/llvm-project/pull/138062
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
https://github.com/dankm updated
https://github.com/llvm/llvm-project/pull/141250
>From 1478c326a7e60b270f2a81e373c390183d1483ed Mon Sep 17 00:00:00 2001
From: Dan McGregor
Date: Fri, 23 May 2025 10:19:22 -0600
Subject: [PATCH] [Clang] Remap paths in OpenMP runtime calls
Apply the debug prefix
@@ -1688,6 +1688,11 @@ def fprofile_sample_accurate : Flag<["-"],
"fprofile-sample-accurate">,
as cold. Otherwise, treat callsites without profile samples as
if
we have no profile}]>,
MarshallingInfoFlag>;
+def fpropeller_profile_use_EQ : Join
Author: Amr Hesham
Date: 2025-05-23T20:54:42+02:00
New Revision: 99f1f0edb54895881bf6ebe5153673ac682092dd
URL:
https://github.com/llvm/llvm-project/commit/99f1f0edb54895881bf6ebe5153673ac682092dd
DIFF:
https://github.com/llvm/llvm-project/commit/99f1f0edb54895881bf6ebe5153673ac682092dd.diff
LO
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/140597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rniwa created
https://github.com/llvm/llvm-project/pull/141277
This PR adds the WebKit checker support for
[[clang::annotate_type("webkit.pointerconversion")]].
When this attribute is set on the return value of a function, the function is
treated as safe to call anywhere an
ilovepi wrote:
### Merge activity
* **May 23, 6:45 PM UTC**: A user started a stack merge that includes this pull
request via
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/141269).
https://github.com/llvm/llvm-project/pull/141269
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Ryosuke Niwa (rniwa)
Changes
This PR adds the WebKit checker support for
[[clang::annotate_type("webkit.pointerconversion")]].
When this attribute is set on the return value of a function, the function is
treated as saf
@@ -1688,6 +1688,11 @@ def fprofile_sample_accurate : Flag<["-"],
"fprofile-sample-accurate">,
as cold. Otherwise, treat callsites without profile samples as
if
we have no profile}]>,
MarshallingInfoFlag>;
+def fpropeller_profile_use_EQ : Join
vsapsai wrote:
> It is concerning to me that we as a company are given **a week** to adjust to
> the change and there is no discussion either about alternative approaches or
> the scale of the changes needed to support this. We do rely on all kinds of
> weird behaviors and side effects in Clan
vsapsai wrote:
> Back to the original issue.
Do you control the module map that specifies a header as private? Does it need
to stay private?
Personally, to me it looks like a bug that a private header is getting picked
up [somewhat accidentally]. I'm not planning to fix this bug, just that re
@@ -1370,10 +1375,14 @@ llvm::Value
*CGOpenMPRuntime::emitUpdateLocation(CodeGenFunction &CGF,
SrcLocStr = OMPBuilder.getOrCreateDefaultSrcLocStr(SrcLocStrSize);
} else {
std::string FunctionName;
+std::string FileName;
if (const auto *FD = dyn_cast_or_null(
https://github.com/mayanez updated
https://github.com/llvm/llvm-project/pull/126961
>From ffb0e0cf837d78b38a4ad9ddebf8cd692682fdc5 Mon Sep 17 00:00:00 2001
From: Miguel Arroyo
Date: Wed, 12 Feb 2025 11:35:10 -0800
Subject: [PATCH] [Clang][CMake][MSVC] Install PDBs alongside executables
---
cl
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/141272
>From 9a8f1f46b1a4e9610b1c3e6d1124ba6321896da9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 23 May 2025 19:41:59 +0200
Subject: [PATCH] [clang][bytecode] Check lifetime of all ptr bas
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/141270
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/evelez7 closed
https://github.com/llvm/llvm-project/pull/140856
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Erick Velez
Date: 2025-05-23T19:35:32Z
New Revision: 441b967ad919de182f54929de701b945a69449c9
URL:
https://github.com/llvm/llvm-project/commit/441b967ad919de182f54929de701b945a69449c9
DIFF:
https://github.com/llvm/llvm-project/commit/441b967ad919de182f54929de701b945a69449c9.diff
LOG: [
@@ -553,7 +553,10 @@ void
WarningsSpecialCaseList::processSections(DiagnosticsEngine &Diags) {
// Each section has a matcher with that section's name, attached to that
// line.
const auto &DiagSectionMatcher = Entry.SectionMatcher;
-unsigned DiagLine = DiagSect
Author: Miguel A. Arroyo
Date: 2025-05-23T12:58:56-07:00
New Revision: 795e88903721e89e105338e1dd464a834156aac7
URL:
https://github.com/llvm/llvm-project/commit/795e88903721e89e105338e1dd464a834156aac7
DIFF:
https://github.com/llvm/llvm-project/commit/795e88903721e89e105338e1dd464a834156aac7.di
https://github.com/mayanez closed
https://github.com/llvm/llvm-project/pull/126961
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/balazske edited
https://github.com/llvm/llvm-project/pull/141076
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Balázs Kéri
Date: 2025-05-23T09:26:31+02:00
New Revision: be50ada9d008673a041a5e675f9d9d70a4572aaa
URL:
https://github.com/llvm/llvm-project/commit/be50ada9d008673a041a5e675f9d9d70a4572aaa
DIFF:
https://github.com/llvm/llvm-project/commit/be50ada9d008673a041a5e675f9d9d70a4572aaa.diff
L
https://github.com/balazske closed
https://github.com/llvm/llvm-project/pull/141076
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/albus-droid updated
https://github.com/llvm/llvm-project/pull/139986
>From 3cc8334092853442f85c5a17a3bd31e373f30da8 Mon Sep 17 00:00:00 2001
From: albus-droid
Date: Wed, 14 May 2025 15:49:11 -0400
Subject: [PATCH 1/8] Added two conditions to check for width of the factor
var
cor3ntin wrote:
I've been considering similar issues recently.
e.g https://compiler-explorer.com/z/5xcKf8Kj1
A more general solution would be to emit diagnostics in system headers if one
of the involved types/expressions names an entity declared outside of a system
header.
I haven't had a bri
mylai-mtk wrote:
Update: Rebased to `main` trunk; keep function signature in LLVM IR
https://github.com/llvm/llvm-project/pull/111661
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Daan De Meyer (DaanDeMeyer)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/141216.diff
2 Files Affected:
- (modified)
clang-tools-extra/clang-include-fixer/find-all-symbols/STLPostfixHeaderMap.cpp
(+1)
- (modified
https://github.com/OCHyams edited
https://github.com/llvm/llvm-project/pull/134643
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/OCHyams ready_for_review
https://github.com/llvm/llvm-project/pull/134643
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/OCHyams updated
https://github.com/llvm/llvm-project/pull/134641
>From ae032c5dd3537662508d77bbb447808f52481f5d Mon Sep 17 00:00:00 2001
From: Orlando Cazalet-Hyams
Date: Tue, 1 Apr 2025 11:59:24 +0100
Subject: [PATCH 01/33] [KeyInstr][Clang] Add ApplyAtomGroup
This is a sco
https://github.com/mylai-mtk updated
https://github.com/llvm/llvm-project/pull/111661
>From b3a7b9f4bd6d65f837b3cc9e4a0e0bd6fdede6ab Mon Sep 17 00:00:00 2001
From: Ming-Yi Lai
Date: Fri, 31 May 2024 17:03:04 +0800
Subject: [PATCH] [clang][RISCV] Emit RISCV function-signature-based CFI label
in
@@ -282,4 +283,25 @@ static_assert(S().SizeOfT() == sizeof(short
*), "");
} // namespace GH68490
+namespace GH83608 {
+
+class single;
+
+class check_constructible {
+ // This makes it a non-aggregate in C++20+.
cor3ntin wrote:
```suggestion
// This make
https://github.com/cor3ntin approved this pull request.
Thanks
https://github.com/llvm/llvm-project/pull/141207
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
⚠️ We detected that you are using a GitHub private e-mail address to contribute
to the repo. Please turn off [Keep my email addresses
private](https://github.com/settings/emails) setting in your account. See
[LLVM
Discourse](https://discourse.llvm.org/t/hidden-email
https://github.com/carlosgalvezp created
https://github.com/llvm/llvm-project/pull/141209
It is deemed more of an annoyance than a help, since the patterns the check
warns about are quite common and idiomatic in C, and there are no good
alternatives.
Thus, enable the check only for C++, where
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Carlos Galvez (carlosgalvezp)
Changes
It is deemed more of an annoyance than a help, since the patterns the check
warns about are quite common and idiomatic in C, and there are no good
alternatives.
Thus, enable the check onl
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Carlos Galvez (carlosgalvezp)
Changes
It is deemed more of an annoyance than a help, since the patterns the check
warns about are quite common and idiomatic in C, and there are no good
alternatives.
Thus, enable the check only for C
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`llvm-clang-x86_64-gcc-ubuntu` running on `sie-linux-worker3` while building
`clang` at step 6 "test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/174/builds/18228
Her
Author: Jiefeng Wang
Date: 2025-05-23T10:38:35+02:00
New Revision: 4f844e7881d86971cd31401d02a418235d18f456
URL:
https://github.com/llvm/llvm-project/commit/4f844e7881d86971cd31401d02a418235d18f456
DIFF:
https://github.com/llvm/llvm-project/commit/4f844e7881d86971cd31401d02a418235d18f456.diff
cor3ntin wrote:
Does this fixes https://github.com/llvm/llvm-project/issues/62444 ?
https://github.com/llvm/llvm-project/pull/141207
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -282,4 +283,25 @@ static_assert(S().SizeOfT() == sizeof(short
*), "");
} // namespace GH68490
+namespace GH83608 {
+
+class single;
+
+class check_constructible {
+ // This makes it a non-aggregate in C++20+.
+ check_constructible() = default;
cor3ntin
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/141207
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
@charan-003 Oliver told me you were happy with him taking over in #141148. Can
you confirm?
I apologize that we incorrectly identified this bug as a good first issue when
it clearly wasn't. Were you able to find other issues to work on?
https://github.com/llvm/llvm-project/pull
https://github.com/OCHyams closed
https://github.com/llvm/llvm-project/pull/134640
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DaanDeMeyer created
https://github.com/llvm/llvm-project/pull/141216
None
>From a6dd3873ef01268f5466bdeebf2bdcc4c314158e Mon Sep 17 00:00:00 2001
From: Daan De Meyer
Date: Tue, 20 May 2025 15:46:29 +0200
Subject: [PATCH] [clang-tools-extra] Add include mappings for
---
..
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Daan De Meyer (DaanDeMeyer)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/141216.diff
2 Files Affected:
- (modified)
clang-tools-extra/clang-include-fixer/find-all-symbols/STLPostfixHeaderMap.cpp
(+1)
OCHyams wrote:
Sorry to all that were added... grappling with messed-up graphite internal
state, GitHub enjoys adding reviewers for me when I change base.
https://github.com/llvm/llvm-project/pull/134638
___
cfe-commits mailing list
cfe-commits@lists.
Author: Orlando Cazalet-Hyams
Date: 2025-05-23T10:02:26+01:00
New Revision: 5c3a99760274a06f8cb7e7247ce69c2fde5fbf2a
URL:
https://github.com/llvm/llvm-project/commit/5c3a99760274a06f8cb7e7247ce69c2fde5fbf2a
DIFF:
https://github.com/llvm/llvm-project/commit/5c3a99760274a06f8cb7e7247ce69c2fde5fbf
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/141207
We check the accessibility of constructors when initializing a default argument
whose type is not an aggregate.
Make sure the check is performed within the correct DeclContext. Otherwise, it
will be delayed un
https://github.com/uweigand approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141110
___
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 `hip-third-party-libs-test`
running on `ext_buildbot_hw_05-hip-docker` while building `clang` at step 4
"annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/206/builds/725
Here is the relevant
@@ -20,10 +20,10 @@ void test_add_template() {
// CHECK: %call = call noundef <10 x float>
@_Z3addIfLj2ELj5EEN8MyMatrixIT_XT0_EXT1_EE8matrix_tERS2_S4_(ptr noundef nonnull
align 4 dereferenceable(40) %Mat1, ptr noundef nonnull align 4
dereferenceable(40) %Mat2)
//
https://github.com/tom-anders closed
https://github.com/llvm/llvm-project/pull/141052
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tom-anders created
https://github.com/llvm/llvm-project/pull/141208
Sorry again about the 3rd PR for this now, but Github won't let me reopen the
2nd one now either... I think it's because I pushed from a different machine
when fixing your comment about the `ranges`. The ne
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Tom Praschan (tom-anders)
Changes
Sorry again about the 3rd PR for this now, but Github won't let me reopen the
2nd one now either... I think it's because I pushed from a different machine
when fixing your comment about the `
higher-performance wrote:
@efriedma-quic: I'm not sure what the appropriate way to test this is. The file
[clang/test/SemaCXX/uninitialized.cpp](https://github.com/llvm/llvm-project/blob/main/clang/test/SemaCXX/uninitialized.cpp)
doesn't include any system headers. Should I just add a system he
@@ -2916,9 +2916,30 @@ CodeGenFunction::EmitLoadOfReference(LValue RefLVal,
llvm::LoadInst *Load =
Builder.CreateLoad(RefLVal.getAddress(), RefLVal.isVolatile());
CGM.DecorateInstructionWithTBAA(Load, RefLVal.getTBAAInfo());
- return makeNaturalAddressForPointer(Load
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-ppc64-aix` running
on `aix-ppc64` while building `clang` at step 6
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/64/builds/3757
Here is the relevant piece
CarlosAlbertoEnciso wrote:
@tromey You added a note about a missing test for the case where a class is
defined inside a function.
That case is handled by
`clang/test/CodeGenCXX/vtable-debug-info-inheritance-simple-main.cpp`
https://github.com/llvm/llvm-project/pull/130255
https://github.com/OCHyams edited
https://github.com/llvm/llvm-project/pull/134638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
101 - 200 of 341 matches
Mail list logo
