https://github.com/davemgreen updated
https://github.com/llvm/llvm-project/pull/135064
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,san
https://github.com/AaronBallman updated
https://github.com/llvm/llvm-project/pull/138271
>From 6360cf51134d978b1ee8b6615128922eb3726d97 Mon Sep 17 00:00:00 2001
From: Aaron Ballman
Date: Fri, 2 May 2025 07:35:57 -0400
Subject: [PATCH 1/4] Handle null pointer constants properly
---
clang/lib/S
https://github.com/AaronBallman updated
https://github.com/llvm/llvm-project/pull/138271
>From 6360cf51134d978b1ee8b6615128922eb3726d97 Mon Sep 17 00:00:00 2001
From: Aaron Ballman
Date: Fri, 2 May 2025 07:35:57 -0400
Subject: [PATCH 1/3] Handle null pointer constants properly
---
clang/lib/S
@@ -13958,7 +13979,11 @@ QualType Sema::CheckAssignmentOperands(Expr *LHSExpr,
ExprResult &RHS,
LHSType->isObjCObjectPointerType(
ConvTy = AssignConvertType::Compatible;
+<<< HEAD
AaronBallman wrote:
Yeah, I just hit that error in my
Author: Aaron Ballman
Date: 2025-05-02T09:19:21-04:00
New Revision: a4ceac7e3e04c32cb3e334eb89b54d8e99a298f8
URL:
https://github.com/llvm/llvm-project/commit/a4ceac7e3e04c32cb3e334eb89b54d8e99a298f8
DIFF:
https://github.com/llvm/llvm-project/commit/a4ceac7e3e04c32cb3e334eb89b54d8e99a298f8.diff
https://github.com/DaanDeMeyer updated
https://github.com/llvm/llvm-project/pull/137840
>From 7e2463f6fb9a3d64b1be25f2ea6ea91de2d4e982 Mon Sep 17 00:00:00 2001
From: Daan De Meyer
Date: Tue, 29 Apr 2025 18:26:36 +0200
Subject: [PATCH] clang-format: Add IncludeSortKey option
Sorting by stem giv
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/132939
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -13958,7 +13979,11 @@ QualType Sema::CheckAssignmentOperands(Expr *LHSExpr,
ExprResult &RHS,
LHSType->isObjCObjectPointerType(
ConvTy = AssignConvertType::Compatible;
+<<< HEAD
erichkeane wrote:
bad merge! Below too
https://githu
https://github.com/diggerlin updated
https://github.com/llvm/llvm-project/pull/137670
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans
@@ -250,6 +250,32 @@ static KeywordStatus getKeywordStatus(const LangOptions
&LangOpts,
return CurStatus;
}
+static bool IsKeywordInCpp(unsigned Flags) {
+ while (Flags != 0) {
+unsigned CurFlag = Flags & ~(Flags - 1);
+Flags = Flags & ~CurFlag;
+switch (static
AaronBallman wrote:
> I'm not sure I buy the justification
FWIW, this came from a post-commit review comment:
https://github.com/llvm/llvm-project/pull/137166#discussion_r2070615023
>, and the flag-diag there is perhaps 'cute' (and frankly, if we find ourself
>doing that again, I would vastly
alexfh wrote:
Please take a look, so far it seems like it should be reverted.
https://github.com/llvm/llvm-project/pull/136203
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
erichkeane wrote:
> > I'm not sure I buy the justification
>
> FWIW, this came from a post-commit review comment: [#137166
> (comment)](https://github.com/llvm/llvm-project/pull/137166#discussion_r2070615023)
>
> > , and the flag-diag there is perhaps 'cute' (and frankly, if we find
> > ourse
zimirza wrote:
> > `localtime_s` was removed from #110363, and will be added to a new pull
> > request and `ctime_s` has not been merged yet (#110676). It might be a good
> > wait before merging this pull request.
>
> I think it is better to remove `localtime_s` and `ctime_s` from this pull
>
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/138271
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -250,6 +250,32 @@ static KeywordStatus getKeywordStatus(const LangOptions
&LangOpts,
return CurStatus;
}
+static bool IsKeywordInCpp(unsigned Flags) {
+ while (Flags != 0) {
+unsigned CurFlag = Flags & ~(Flags - 1);
+Flags = Flags & ~CurFlag;
+switch (static
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/138245
>From 63aedb490ecdb251a5005f40e8e1bd3dd2f89e70 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Fri, 2 May 2025 10:53:38 +0200
Subject: [PATCH 1/2] [Clang] Implement CWG3005 Function parameters should
never
@@ -623,6 +623,8 @@ Bug Fixes to C++ Support
- Clang now emits a warning when class template argument deduction for alias
templates is used in C++17. (#GH133806)
- Fix a crash when checking the template template parameters of a dependent
lambda appearing in an alias declaratio
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/138247
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Aaron Ballman
Date: 2025-05-02T09:39:20-04:00
New Revision: 0b2ab11321d2b400987e9ade28a221dff67aea7d
URL:
https://github.com/llvm/llvm-project/commit/0b2ab11321d2b400987e9ade28a221dff67aea7d
DIFF:
https://github.com/llvm/llvm-project/commit/0b2ab11321d2b400987e9ade28a221dff67aea7d.diff
rorth wrote:
/cherry-pick e71c8ea3cc73c8f7b0382468f355a254166d3a72
https://github.com/llvm/llvm-project/pull/137141
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rorth milestoned
https://github.com/llvm/llvm-project/pull/137141
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rorth wrote:
This patch is required to build `flang` on Solaris/amd64. Until very recently,
a 2-stage build of `main` with `flang` included worked with `clang-20` as build
compiler since `flang` was only built in stage 2 and the freshly built
`clang-21` does include the patch.. However, now
@@ -934,7 +934,7 @@ void tools::addLTOOptions(const ToolChain &ToolChain, const
ArgList &Args,
std::optional OptVal =
llvm::StringSwitch>(ArgVecLib->getValue())
.Case("Accelerate", "Accelerate")
-.Case("LIBMVEC", "LIBMVEC-X86")
+
https://github.com/tigbr created
https://github.com/llvm/llvm-project/pull/138295
This patch improves the performance of the Clang Static Analyzer by passing
separate `BumpPtrAllocator` instances to its data structures instead of using
the `BumpPtrAllocator` of the ExplodedGraph everywhere.
I
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: None (tigbr)
Changes
This patch improves the performance of the Clang Static Analyzer by passing
separate `BumpPtrAllocator` instances to its data structures instead of using
the `BumpPtrAllocator` of the ExplodedGraph e
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (tigbr)
Changes
This patch improves the performance of the Clang Static Analyzer by passing
separate `BumpPtrAllocator` instances to its data structures instead of using
the `BumpPtrAllocator` of the ExplodedGraph everywhere.
It gre
@@ -1987,7 +1987,8 @@ void CGDebugInfo::CollectRecordFields(
// Bump field number for next field.
++fieldNo;
- } else if (CGM.getCodeGenOpts().EmitCodeView) {
+ } else if (CGM.getCodeGenOpts().hasMaybeUnusedDebugInfo() ||
+ CGM.getCode
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/include/clang/StaticAnalyzer/Core/PathSensiti
https://github.com/AlexVlx updated
https://github.com/llvm/llvm-project/pull/134016
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-s
vbvictor wrote:
Did you run code sanitizer over all clang-tidy code or you only interested in
this particular file?
I think there are many more places where such `nullptr` dereference can happen,
e.g.
https://github.com/llvm/llvm-project/blob/52d2b589b2d7adc3e2b4ea503df4131e088de61f/clang-tools
@@ -428,6 +429,52 @@ mlir::LogicalResult CIRGenFunction::emitBreakStmt(const
clang::BreakStmt &s) {
return mlir::success();
}
+const CaseStmt *CIRGenFunction::foldCaseStmt(const clang::CaseStmt &s,
+ mlir::Type condType,
+
https://github.com/MacDue approved this pull request.
LGTM, thanks :+1:
There's one other use of "LIBMVEC" in flang that looks suspicious, but I've not
check if it's actually an issue:
https://github.com/llvm/llvm-project/blob/d6dbe7799e638702309e23fc7b73d4be2231e2ac/flang/lib/Frontend/Compil
https://github.com/yxsamliu updated
https://github.com/llvm/llvm-project/pull/138162
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,sans-
https://github.com/yxsamliu edited
https://github.com/llvm/llvm-project/pull/138162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/138107
Rate limit · GitHub
body {
background-color: #f6f8fa;
color: #24292e;
font-family: -apple-system,BlinkMacSystemFont,Segoe
UI,Helvetica,Arial,s
zahiraam wrote:
> This is not needed IMO. This is a simple matcher, `op` is expected to always
> be bound.
>
> One might say that it can't hurt to add extra checks, but I don't think
> that's good justification for adding more code / bloat, even if in this case
> it's a single line. Still, 5%
https://github.com/AmrDeveloper approved this pull request.
https://github.com/llvm/llvm-project/pull/138222
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yxsamliu edited
https://github.com/llvm/llvm-project/pull/138162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2641,6 +2641,28 @@ def int_amdgcn_perm :
// GFX9 Intrinsics
//===--===//
+/// This is a general-purpose intrinsic for all operations that take a pointer
+/// a base location in LDS, and a data size and us
@@ -428,6 +429,52 @@ mlir::LogicalResult CIRGenFunction::emitBreakStmt(const
clang::BreakStmt &s) {
return mlir::success();
}
+const CaseStmt *CIRGenFunction::foldCaseStmt(const clang::CaseStmt &s,
+ mlir::Type condType,
+
https://github.com/yxsamliu edited
https://github.com/llvm/llvm-project/pull/138162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/andykaylor updated
https://github.com/llvm/llvm-project/pull/138222
>From b3ae2d60a178f3c5bb8950804041386a667abc6e Mon Sep 17 00:00:00 2001
From: Andy Kaylor
Date: Tue, 29 Apr 2025 15:47:01 -0700
Subject: [PATCH 1/3] [CIR] Refactor global variable emission and
initialization
Author: cor3ntin
Date: 2025-05-02T19:36:53+02:00
New Revision: e59ed0fb0f3324ae3fce4080e15472a316713dfd
URL:
https://github.com/llvm/llvm-project/commit/e59ed0fb0f3324ae3fce4080e15472a316713dfd
DIFF:
https://github.com/llvm/llvm-project/commit/e59ed0fb0f3324ae3fce4080e15472a316713dfd.diff
LOG:
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/138291
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Vlad Serebrennikov
Date: 2025-05-02T10:48:08+03:00
New Revision: 140c2b6d081b781b19f50bf1c4fbf4b17bfacdc1
URL:
https://github.com/llvm/llvm-project/commit/140c2b6d081b781b19f50bf1c4fbf4b17bfacdc1
DIFF:
https://github.com/llvm/llvm-project/commit/140c2b6d081b781b19f50bf1c4fbf4b17bfacdc1.
https://github.com/mstorsjo edited
https://github.com/llvm/llvm-project/pull/138120
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mstorsjo closed
https://github.com/llvm/llvm-project/pull/138120
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mstorsjo approved this pull request.
I updated the PR description with the details I wanted to have included in the
commit message.
https://github.com/llvm/llvm-project/pull/138120
___
cfe-commits mailing list
cfe-commits@lists.llvm
Author: Mateusz Mikuła
Date: 2025-05-02T11:07:15+03:00
New Revision: abe93fe7c88c477343c884036982ddc15f820425
URL:
https://github.com/llvm/llvm-project/commit/abe93fe7c88c477343c884036982ddc15f820425
DIFF:
https://github.com/llvm/llvm-project/commit/abe93fe7c88c477343c884036982ddc15f820425.diff
https://github.com/mstorsjo approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/138217
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mstorsjo wrote:
> Okay, you were right. Keeping this in a single description hurts readability
> a lot.
>
> So I have opened a series of smaller PRs: #138117 #138118 #138119 #138120
>
> This branch is equal to the other ones (once they are merged together), so we
> can go either way. I think
Author: Vlad Serebrennikov
Date: 2025-05-02T11:02:01+03:00
New Revision: db247ddc9d5411ace70b478f7119eb44a43a8e5c
URL:
https://github.com/llvm/llvm-project/commit/db247ddc9d5411ace70b478f7119eb44a43a8e5c
DIFF:
https://github.com/llvm/llvm-project/commit/db247ddc9d5411ace70b478f7119eb44a43a8e5c.
Author: Mateusz Mikuła
Date: 2025-05-02T11:01:37+03:00
New Revision: 592243c1cb3ea53b34033132a87b0d14af9d1079
URL:
https://github.com/llvm/llvm-project/commit/592243c1cb3ea53b34033132a87b0d14af9d1079
DIFF:
https://github.com/llvm/llvm-project/commit/592243c1cb3ea53b34033132a87b0d14af9d1079.diff
https://github.com/mstorsjo closed
https://github.com/llvm/llvm-project/pull/138118
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/137958
___
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 `ml-opt-dev-x86-64` running
on `ml-opt-dev-x86-64-b2` while building `clang,llvm,mlir` at step 4
"cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/137/builds/17828
Here is the relevan
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `llvm-clang-aarch64-darwin`
running on `doug-worker-4` while building `clang` at step 6
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/190/builds/19356
Here is th
Author: Mateusz Mikuła
Date: 2025-05-02T11:02:11+03:00
New Revision: 9bc20fbbadb18a1c3415d4442066ed35a6bcc175
URL:
https://github.com/llvm/llvm-project/commit/9bc20fbbadb18a1c3415d4442066ed35a6bcc175
DIFF:
https://github.com/llvm/llvm-project/commit/9bc20fbbadb18a1c3415d4442066ed35a6bcc175.diff
https://github.com/mstorsjo closed
https://github.com/llvm/llvm-project/pull/138119
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Vlad Serebrennikov
Date: 2025-05-02T11:24:08+03:00
New Revision: d0dcfd4c68171c3b9b863626c08efa7f38cd0c54
URL:
https://github.com/llvm/llvm-project/commit/d0dcfd4c68171c3b9b863626c08efa7f38cd0c54
DIFF:
https://github.com/llvm/llvm-project/commit/d0dcfd4c68171c3b9b863626c08efa7f38cd0c54.
Keenuts wrote:
Closed in favor of https://github.com/llvm/llvm-project/pull/135794
https://github.com/llvm/llvm-project/pull/134844
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Keenuts closed
https://github.com/llvm/llvm-project/pull/134844
___
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
`sanitizer-x86_64-linux-android` running on `sanitizer-buildbot-android` while
building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/186/builds/8689
Here is the releva
ostannard wrote:
I'd assumed that this was a sufficiently uncommon edge-case that it wasn't
worth a release note, but I'll keep that in mind in future and add release
notes for things like this.
https://github.com/llvm/llvm-project/pull/126774
___
cf
mati865 wrote:
Thank you for taking care of it!
https://github.com/llvm/llvm-project/pull/134494
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mati865 closed
https://github.com/llvm/llvm-project/pull/134494
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mati865 wrote:
Looks good, thanks.
https://github.com/llvm/llvm-project/pull/138120
___
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-gcc-ubuntu-no-asserts` running on `doug-worker-6` while
building `clang` at step 6 "test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/202/builds/1014
https://github.com/topperc closed
https://github.com/llvm/llvm-project/pull/138324
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Craig Topper
Date: 2025-05-02T16:10:18-07:00
New Revision: 123758b1f4a8106926d95268ea8dc27158b6393a
URL:
https://github.com/llvm/llvm-project/commit/123758b1f4a8106926d95268ea8dc27158b6393a
DIFF:
https://github.com/llvm/llvm-project/commit/123758b1f4a8106926d95268ea8dc27158b6393a.diff
https://github.com/vsapsai created
https://github.com/llvm/llvm-project/pull/138358
Restore the behavior existing prior to
fe2eefc4718f57e1753f7bd51c158fc03d70b34f. Make reporting of unevaluated
directive source range more consistent and with fewer assumptions. In case of a
failed evaluation
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Volodymyr Sapsai (vsapsai)
Changes
Restore the behavior existing prior to
fe2eefc4718f57e1753f7bd51c158fc03d70b34f. Make reporting of unevaluated
directive source range more consistent and with fewer assumptions. In case of a
failed eval
@@ -138,6 +142,12 @@ let Attributes = [Const, NoThrow,
RequiredVectorWidth<256>], Features = "avx" in
}
}
+// PRFCHW
+let Features = "prfchw", Header = "x86intrin.h", Attributes = [NoThrow, Const]
in {
+ def _m_prefetch : X86LibBuiltin<"void(void *)">;
rn
https://github.com/Ankur-0429 updated
https://github.com/llvm/llvm-project/pull/136807
>From 44a384b8ad774939c7664c8a476e852f1d4a7341 Mon Sep 17 00:00:00 2001
From: Ankur Ahir
Date: Tue, 22 Apr 2025 20:04:34 -0700
Subject: [PATCH 01/13] Upstream enum support
---
clang/lib/CIR/CodeGen/CIRGenMo
@@ -2641,6 +2641,28 @@ def int_amdgcn_perm :
// GFX9 Intrinsics
//===--===//
+/// This is a general-purpose intrinsic for all operations that take a pointer
+/// a base location in LDS, and a data size and us
https://github.com/shafik created
https://github.com/llvm/llvm-project/pull/138352
Static analysis flagged the use of Detail because we were not using std::move
when returning values. Modified the returns to use std::move.
>From f12ee9b85d6f457061b7976ef929e3547c510b06 Mon Sep 17 00:00:00 2001
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 --
clang/include/clang-c/Platform.h
``
View t
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/138346
>From 821d56db431ec2587eefc18f9f9dab5f5dc48c7e Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Fri, 2 May 2025 23:12:34 +0200
Subject: [PATCH 1/2] [CIR] Upstream local VectorType with 0 init elemnts
---
@@ -70,12 +72,15 @@ void foo() {
// CIR: %[[VEC_E:.*]] = cir.alloca !cir.vector<4 x !s32i>,
!cir.ptr>, ["e", init]
// CIR: %[[VEC_F:.*]] = cir.alloca !cir.vector<4 x !s32i>,
!cir.ptr>, ["f", init]
// CIR: %[[VEC_G:.*]] = cir.alloca !cir.vector<4 x !s32i>,
!cir.ptr>, ["g", in
https://github.com/andykaylor created
https://github.com/llvm/llvm-project/pull/138368
This change adds additional checks to a few places where a simple struct in C++
code was triggering `errorNYI` in places where no additional handling was
needed, and adds a very small amount of trivial initi
llvmbot wrote:
@llvm/pr-subscribers-clangir
Author: Andy Kaylor (andykaylor)
Changes
This change adds additional checks to a few places where a simple struct in C++
code was triggering `errorNYI` in places where no additional handling was
needed, and adds a very small amount of trivial i
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Andy Kaylor (andykaylor)
Changes
This change adds additional checks to a few places where a simple struct in C++
code was triggering `errorNYI` in places where no additional handling was
needed, and adds a very small amount of trivial ini
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-x86_64-linux-bootstrap-msan` running on `sanitizer-buildbot6` while
building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/164/builds/9589
Here is the releva
@@ -52,8 +54,17 @@ intCastExpression(bool IsSigned,
const auto StaticCastExpr = cxxStaticCastExpr(has(ImplicitCastExpr));
const auto FunctionalCastExpr = cxxFunctionalCastExpr(has(ImplicitCastExpr));
+ // Match function calls or variable references not directly wrapped by
https://github.com/RiverDave updated
https://github.com/llvm/llvm-project/pull/134188
>From f5f0b14c9f49b2b956e6f1c63f24e203d6f4beef Mon Sep 17 00:00:00 2001
From: David Rivera
Date: Wed, 2 Apr 2025 21:02:00 -0400
Subject: [PATCH] [clang-tidy] Improve integer comparison by matching valid
expre
https://github.com/ojhunt created
https://github.com/llvm/llvm-project/pull/138376
When parsing a function pointer typed field we use
Parser::ParseTypeQualifierListOpt, but then drops subsequent type attributes
and qualifiers unless explicitly handled.
There is an existing solution for the _A
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Oliver Hunt (ojhunt)
Changes
When parsing a function pointer typed field we use
Parser::ParseTypeQualifierListOpt, but then drops subsequent type attributes
and qualifiers unless explicitly handled.
There is an existing solution for the
https://github.com/RiverDave updated
https://github.com/llvm/llvm-project/pull/131969
>From ed0c4bd4c5200819b82e97dc8032b7defdfd88da Mon Sep 17 00:00:00 2001
From: David Rivera
Date: Sun, 16 Mar 2025 16:20:16 -0400
Subject: [PATCH] [clang-tidy] Add support for Initialization Forwarding in
Nest
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/138377
In consteval calls rewriting, we expect an instantiated AST to remain
untouched. However, this was not true when rebuilding a MemberExpr in
TreeTransform, as it relies on FoundDecl to build a resolved member ca
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Younan Zhang (zyn0217)
Changes
In consteval calls rewriting, we expect an instantiated AST to remain
untouched. However, this was not true when rebuilding a MemberExpr in
TreeTransform, as it relies on FoundDecl to build a resolved member
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/136828
>From d51ed1b628eac11e3ff1a2bb5fffd87e7ca127ea Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Wed, 30 Apr 2025 22:26:59 -0700
Subject: [PATCH] [clang][ptrauth] add support for options parameter to
__ptrauth
T
@@ -360,6 +360,13 @@ class Sema;
LLVM_PREFERRED_TYPE(bool)
unsigned ObjCLifetimeConversionBinding : 1;
+/// Whether the source expression was originally a single element
+/// braced-init-list. Such a conversion is not a perfect match,
+/// as we prefer a st
@@ -412,6 +419,12 @@ class Sema;
bool isPerfect(const ASTContext &C) const {
if (!isIdentityConversion())
return false;
+
+ // We might prefer a std::initializer constructor,
zyn0217 wrote:
`std::initializer_list`
https://github.com/llv
RiverDave wrote:
Ping
https://github.com/llvm/llvm-project/pull/131969
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/137580
>From 9c29520abea852086f8f4c555e2ad1ea7c711800 Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Sun, 27 Apr 2025 22:33:44 -0700
Subject: [PATCH] [clang] Add `__ptrauth_restricted_intptr` qualifier
__ptrauth_rest
@@ -9284,6 +9284,12 @@ void LinkerWrapper::ConstructJob(Compilation &C, const
JobAction &JA,
CmdArgs.push_back(Args.MakeArgString(
"--device-linker=" + TC->getTripleString() + "=" + Arg));
+ // Enable internalization for AMDGPU.
+ if (TC->getTrip
pinskia wrote:
Just FYI, the GCC name of the option is -Wjump-misses-init .
https://github.com/llvm/llvm-project/pull/138009
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/topperc created
https://github.com/llvm/llvm-project/pull/138378
RISC-V with -mrvv-vector-bits-min supports giving a size to our scalable vector
types. To do this, we represent the vector as a fixed vector in memory and need
to cast back and force to scable vectors.
For i1
cor3ntin wrote:
I'm fairly opposed to adding this flag before Sofia - My assumption is that
this is a C++26 feature (and one that is backward compatible) - and we should
not add a flag for every standard feature. _if_ the committee fails to land
this in Sofia, then we can add the flag before t
501 - 600 of 604 matches
Mail list logo
