https://github.com/fzou1 closed https://github.com/llvm/llvm-project/pull/141486
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Feng Zou
Date: 2025-05-27T14:58:32+08:00
New Revision: 9e6fc8dedd75bcb9a389921bee12d18eecbd9908
URL:
https://github.com/llvm/llvm-project/commit/9e6fc8dedd75bcb9a389921bee12d18eecbd9908
DIFF:
https://github.com/llvm/llvm-project/commit/9e6fc8dedd75bcb9a389921bee12d18eecbd9908.diff
LOG:
https://github.com/balazs-benics-sonarsource updated
https://github.com/llvm/llvm-project/pull/140924
From 084d821b62d5de473d32d3506da95fdd7bad1cfe Mon Sep 17 00:00:00 2001
From: Balazs Benics
Date: Thu, 15 May 2025 17:20:29 +0200
Subject: [PATCH 1/7] [analyzer] Introduce the check::BlockEntran
@@ -166,6 +179,23 @@ class CheckerDocumentation
/// check::Bind
void checkBind(SVal Loc, SVal Val, const Stmt *S, CheckerContext &) const {}
+ /// Called after a CFG edge is taken within a function.
+ ///
+ /// This callback can be used to obtain information about poten
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/141547
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -35,6 +35,169 @@ static void populateMemberTypeInfo(RecordInfo &I,
AccessSpecifier &Access,
const DeclaratorDecl *D,
bool IsStatic = false);
+static void getTemplateParameters(const TemplateParameterList
kawashima-fj wrote:
I've confirmed the result of Fujitsu Compiler Test Suite. The only correctness
issue affected by this commit is
https://github.com/fujitsu/compiler-test-suite/blob/main/Fortran/0347/0347_0240.f,
which will be resolved by #140709.
https://github.com/llvm/llvm-project/pull/1
https://github.com/4vtomat created
https://github.com/llvm/llvm-project/pull/141548
Currently we only check the required features passed by command line arguments.
We also need to check the features passed by using target features.
>From a125a7137fcfb05418e97d2823d5b0a7f099645d Mon Sep 17 00:0
@@ -35,6 +35,169 @@ static void populateMemberTypeInfo(RecordInfo &I,
AccessSpecifier &Access,
const DeclaratorDecl *D,
bool IsStatic = false);
+static void getTemplateParameters(const TemplateParameterList
@@ -35,6 +35,169 @@ static void populateMemberTypeInfo(RecordInfo &I,
AccessSpecifier &Access,
const DeclaratorDecl *D,
bool IsStatic = false);
+static void getTemplateParameters(const TemplateParameterList
https://github.com/petrhosek deleted
https://github.com/llvm/llvm-project/pull/138065
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/petrhosek edited
https://github.com/llvm/llvm-project/pull/138065
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/petrhosek edited
https://github.com/llvm/llvm-project/pull/138065
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/141474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyn0217 created
https://github.com/llvm/llvm-project/pull/141547
I think the intent of df18ee96206 was to substitute only those non-packs into a
pack expansion type (e.g. `T` in `T::pack`...), so let's hold off pack
expansions explicitly, in case there are calls coming from
@@ -35,6 +35,169 @@ static void populateMemberTypeInfo(RecordInfo &I,
AccessSpecifier &Access,
const DeclaratorDecl *D,
bool IsStatic = false);
+static void getTemplateParameters(const TemplateParameterList
llvmbot wrote:
@llvm/pr-subscribers-backend-risc-v
Author: Brandon Wu (4vtomat)
Changes
Currently we only check the required features passed by command line arguments.
We also need to check the features passed by using target features.
---
Full diff: https://github.com/llvm/llvm-project/
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Brandon Wu (4vtomat)
Changes
Currently we only check the required features passed by command line arguments.
We also need to check the features passed by using target features.
---
Full diff: https://github.com/llvm/llvm-project/pull/1415
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Younan Zhang (zyn0217)
Changes
I think the intent of df18ee96206 was to substitute only those non-packs into a
pack expansion type (e.g. `T` in `T::pack...`), so let's hold off pack
expansions explicitly, in case there are calls coming fr
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/141547
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2025-05-27T08:12:18+02:00
New Revision: 17ef0fe59437f107e8f1cf7edb2fe109de4e332f
URL:
https://github.com/llvm/llvm-project/commit/17ef0fe59437f107e8f1cf7edb2fe109de4e332f
DIFF:
https://github.com/llvm/llvm-project/commit/17ef0fe59437f107e8f1cf7edb2fe109de4e332f.diff
L
@@ -141,16 +143,41 @@ std::vector generateMissingIncludeDiagnostics(
AST.getPreprocessor().getHeaderSearchInfo(), MainFile});
llvm::StringRef HeaderRef{Spelling};
-bool Angled = HeaderRef.starts_with("<");
+
+bool IsAngled = false;
kadirce
https://github.com/kadircet requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/140594
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/134375
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kadircet wrote:
LGTM, but cc @llvm/clang-area-team, in case they have any concerns here
https://github.com/llvm/llvm-project/pull/140870
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet approved this pull request.
https://github.com/llvm/llvm-project/pull/139463
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -110,20 +110,81 @@ static std::string computeBaseSysRoot(const Driver &D,
bool IncludeTriple) {
return std::string(SysRootDir);
}
+static bool hasGCCToolChainAlongSideClang(const Driver &D) {
+ SmallString<128> GCCDir;
+ llvm::sys::path::append(GCCDir, D.Dir, "..", D.g
https://github.com/quic-garvgupt updated
https://github.com/llvm/llvm-project/pull/121829
>From 8d6d2e0f1eaa0b81613a786dbe003ddf7d0a2983 Mon Sep 17 00:00:00 2001
From: Garvit Gupta
Date: Fri, 13 Dec 2024 05:31:56 -0800
Subject: [PATCH] [Driver] Teach Barmetal toolchain about GCC installation
T
@@ -110,20 +110,81 @@ static std::string computeBaseSysRoot(const Driver &D,
bool IncludeTriple) {
return std::string(SysRootDir);
}
+static bool hasGCCToolChainAlongSideClang(const Driver &D) {
+ SmallString<128> GCCDir;
+ llvm::sys::path::append(GCCDir, D.Dir, "..", D.g
vsapsai wrote:
Do you include proto headers directly or through the wrapper headers?
Are there any important macros in proto headers?
Do you need to abuse macros to get something out of proto headers? Kinda like
`#define private public` but more realistic.
My initial reaction is to offer a so
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-riscv-rva23-evl-vec-2stage` running on
`rise-clang-riscv-rva23-evl-vec-2stage` while building `clang,llvm` at step 4
"annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/132/builds/1244
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 01/16] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
=?UT
pvelesko wrote:
ping
https://github.com/llvm/llvm-project/pull/136412
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/asavonic approved this pull request.
Good catch, LGTM.
https://github.com/llvm/llvm-project/pull/141474
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
fzou1 wrote:
> Looks like this breaks tests on macOS:
> http://45.33.8.238/macm1/107398/step_6.txt
>
> Please take a look and revert for now if it takes a while to fix.
Thank you. I've reproduced this issue on MacOS and updated PR #141486 to fix it.
https://github.com/llvm/llvm-project/pull/1
https://github.com/fzou1 edited https://github.com/llvm/llvm-project/pull/141486
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/fzou1 edited https://github.com/llvm/llvm-project/pull/141486
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/fzou1 updated
https://github.com/llvm/llvm-project/pull/141486
>From 5f839ebc0d21f393b30821b1df44d066e4b11d2b Mon Sep 17 00:00:00 2001
From: Feng Zou
Date: Mon, 26 May 2025 20:16:34 +0800
Subject: [PATCH 1/2] [Driver][X86] Fix LIT test failure on Solaris OS + AMD64
CPU.
The
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/141518
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2025-05-26T19:43:28-07:00
New Revision: 23c2f8827b13ef8c72f09f931a70e088c35ccc77
URL:
https://github.com/llvm/llvm-project/commit/23c2f8827b13ef8c72f09f931a70e088c35ccc77
DIFF:
https://github.com/llvm/llvm-project/commit/23c2f8827b13ef8c72f09f931a70e088c35ccc77.diff
L
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/141518
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/KanRobert approved this pull request.
https://github.com/llvm/llvm-project/pull/141486
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/140180
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Finn Plummer
Date: 2025-05-26T18:40:59-07:00
New Revision: 02595417cef525cf1eeeb66e4719a3f9e9dc88b0
URL:
https://github.com/llvm/llvm-project/commit/02595417cef525cf1eeeb66e4719a3f9e9dc88b0
DIFF:
https://github.com/llvm/llvm-project/commit/02595417cef525cf1eeeb66e4719a3f9e9dc88b0.diff
s-watanabe314 wrote:
Thank you for confirming.
@MaskRay, do you have any comments regarding the handling of warning messages?
https://github.com/llvm/llvm-project/pull/132680
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 01/16] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
=?UT
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 01/16] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
=?UT
@@ -271,26 +271,32 @@ class CFGWalker {
// translateAttrExpr needs it, but that should be moved too.
class CapabilityExpr {
private:
- /// The capability expression and whether it's negated.
- llvm::PointerIntPair CapExpr;
+ /// The capability expression and flags.
+ llvm::
wenju-he wrote:
> do we know why
> https://github.com/llvm/llvm-project/blob/main/clang/include/clang/Driver/Options.td#L5894-L5901
> doesn't declare _EQ version for cc1?
Git history shows the earliest commit
https://github.com/llvm/llvm-project/commit/f3e624ca73b007552554b31358f4abde9eb2d3b7
https://github.com/wenju-he edited
https://github.com/llvm/llvm-project/pull/140870
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wenju-he edited
https://github.com/llvm/llvm-project/pull/140870
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wenju-he updated
https://github.com/llvm/llvm-project/pull/140870
>From f5e675f17ea737b0668e626f34d013153368425e Mon Sep 17 00:00:00 2001
From: Wenju He
Date: Wed, 21 May 2025 02:08:34 -0700
Subject: [PATCH 1/3] [ClangTool] Use CC1Option flag resource-dir in
injectResourceDi
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/SemaSYCL/sycl-external-attribute.cpp
cl
https://github.com/schittir updated
https://github.com/llvm/llvm-project/pull/140282
>From abdbf8905d324f9b935b34bbc97c508ede5ac028 Mon Sep 17 00:00:00 2001
From: "Chittireddy, Sindhu"
Date: Fri, 16 May 2025 08:51:06 -0700
Subject: [PATCH 1/3] Add sycl_external attribute
---
clang/include/cla
https://github.com/dankm updated
https://github.com/llvm/llvm-project/pull/141250
>From e4fc23b1486180ec01a68f0d2c367d8d15aa8203 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
dankm wrote:
Neat, the original was good, and apparently some tests don't run on my system.
Going back to to that version :/
https://github.com/llvm/llvm-project/pull/141250
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 01/15] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20?=
=?UT
https://github.com/dankm updated
https://github.com/llvm/llvm-project/pull/141250
>From 8e534d8fe14d70e4fa65dc6d8dad8ad4de73caae 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
@@ -43,10 +43,18 @@ internal::Matcher callToGet(const
internal::Matcher &OnClass) {
.bind("redundant_get");
}
-internal::Matcher knownSmartptr() {
+internal::Matcher knownSmartptrAny() {
return recordDecl(hasAnyName("::std::unique_ptr", "::std::shared_ptr"));
}
+i
https://github.com/kadircet approved this pull request.
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/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/140388
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Tommy Chen
Date: 2025-05-27T05:57:51+08:00
New Revision: 4cb25e2d37496a5f52a62a1f411a04dac20a8666
URL:
https://github.com/llvm/llvm-project/commit/4cb25e2d37496a5f52a62a1f411a04dac20a8666
DIFF:
https://github.com/llvm/llvm-project/commit/4cb25e2d37496a5f52a62a1f411a04dac20a8666.diff
LO
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/141524
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2025-05-26T14:57:13-07:00
New Revision: 8d49c64fa27977071ced0ef2bac3f38ac244276f
URL:
https://github.com/llvm/llvm-project/commit/8d49c64fa27977071ced0ef2bac3f38ac244276f
DIFF:
https://github.com/llvm/llvm-project/commit/8d49c64fa27977071ced0ef2bac3f38ac244276f.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/141525
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/141523
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2025-05-26T14:57:09-07:00
New Revision: fe2b921c42349c52c87dc1a4eb79e2452b79e587
URL:
https://github.com/llvm/llvm-project/commit/fe2b921c42349c52c87dc1a4eb79e2452b79e587
DIFF:
https://github.com/llvm/llvm-project/commit/fe2b921c42349c52c87dc1a4eb79e2452b79e587.diff
L
Author: Kazu Hirata
Date: 2025-05-26T14:57:05-07:00
New Revision: 36d57677bbbcc4f32e7aa16b73ca22fcae13a3aa
URL:
https://github.com/llvm/llvm-project/commit/36d57677bbbcc4f32e7aa16b73ca22fcae13a3aa
DIFF:
https://github.com/llvm/llvm-project/commit/36d57677bbbcc4f32e7aa16b73ca22fcae13a3aa.diff
L
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/141410
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Congcong Cai
Date: 2025-05-27T05:56:48+08:00
New Revision: f1d8e37f2cdb7266633743072d885643c82a6e1c
URL:
https://github.com/llvm/llvm-project/commit/f1d8e37f2cdb7266633743072d885643c82a6e1c
DIFF:
https://github.com/llvm/llvm-project/commit/f1d8e37f2cdb7266633743072d885643c82a6e1c.diff
@@ -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/inbelic updated
https://github.com/llvm/llvm-project/pull/140180
>From 05cd843e21cd8d93a94914d13843871886146dab Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Thu, 15 May 2025 22:35:30 +
Subject: [PATCH 1/3] [HLSL][RootSignature] Add parsing infastructure for
StaticSa
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Walter Lee (googlewalt)
Changes
This reverts commit d25f95fdbc5314f30618912e18f00ad4dd720fa0, after 9e306ad4
was rolled back in e2a88553 (r576388).
---
Full diff: https://github.com/llvm/llvm-project/pull/141532.diff
1 Files Affected:
https://github.com/googlewalt closed
https://github.com/llvm/llvm-project/pull/141532
___
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/140180
>From 05cd843e21cd8d93a94914d13843871886146dab Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Thu, 15 May 2025 22:35:30 +
Subject: [PATCH 1/2] [HLSL][RootSignature] Add parsing infastructure for
StaticSa
https://github.com/googlewalt created
https://github.com/llvm/llvm-project/pull/141532
This reverts commit d25f95fdbc5314f30618912e18f00ad4dd720fa0, after 9e306ad4
was rolled back in e2a88553 (r576388).
>From 95dc425f757d1f0a895612bca6aaf8e80f44cf9e Mon Sep 17 00:00:00 2001
From: Walter Lee
D
robincaloudis wrote:
@owenca and @ MyDeveloperDay, do you mind to take a look when you find some
time? Thanks in advance.
https://github.com/llvm/llvm-project/pull/139463
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/c
https://github.com/AlexMaclean approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141278
___
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/140180
>From 05cd843e21cd8d93a94914d13843871886146dab Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Thu, 15 May 2025 22:35:30 +
Subject: [PATCH] [HLSL][RootSignature] Add parsing infastructure for
StaticSample
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/140180
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/141524
These are identified by misc-include-cleaner. I've filtered out those
that break builds. Also, I'm staying away from llvm-config.h,
config.h, and Compiler.h, which likely cause platform- or
compiler-spec
kazutakahirata wrote:
> I only spotchecked, but it should be good if it still builds.
>
> Btw, WDYT of IWYU? Maybe we could improve include hygiene for the static
> analyzer.
@steakhal Thanks for the review! I personally like the IWYU approach, but the
LLVM project uses a different approach
https://github.com/kadircet approved this pull request.
https://github.com/llvm/llvm-project/pull/141410
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
steakhal wrote:
Thanks for sharing your opinion.
I think for long term IWYU should be beneficial for our subsubproject, and may
enable us to see how the include hierarchy looks like and to restructure and
split some headers.
https://github.com/llvm/llvm-project/pull/141525
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/141523
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/141524
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal approved this pull request.
I only spotchecked, but it should be good if it still builds.
Btw, WDYT of IWYU? Maybe we could improve include hygiene for the static
analyzer.
https://github.com/llvm/llvm-project/pull/141525
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
Changes
These are identified by misc-include-cleaner. I've filtered out those
that break builds. Also, I'm staying away from llvm-config.h,
config.h, and Compiler.h, which likely cause platform- or
compiler-sp
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/141525
These are identified by misc-include-cleaner. I've filtered out those
that break builds. Also, I'm staying away from llvm-config.h,
config.h, and Compiler.h, which likely cause platform- or
compiler-spec
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
Changes
These are identified by misc-include-cleaner. I've filtered out those
that break builds. Also, I'm staying away from llvm-config.h,
config.h, and Compiler.h, which likely cause platform- or
compiler-sp
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Kazu Hirata (kazutakahirata)
Changes
These are identified by misc-include-cleaner. I've filtered out those
that break builds. Also, I'm staying away from llvm-config.h,
config.h, and Compiler.h, which likely cause platform- or
compiler-sp
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/141523
These are identified by misc-include-cleaner. I've filtered out those
that break builds. Also, I'm staying away from llvm-config.h,
config.h, and Compiler.h, which likely cause platform- or
compiler-spec
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Kazu Hirata (kazutakahirata)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/141518.diff
1 Files Affected:
- (modified) clang/lib/Format/MacroCallReconstructor.cpp (+2-3)
``diff
diff --git a/clang/lib
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/141518
None
>From 363881869140e8ce4874a91b9b46d64a9647a7a1 Mon Sep 17 00:00:00 2001
From: Kazu Hirata
Date: Mon, 26 May 2025 09:43:03 -0700
Subject: [PATCH] [Format] Use llvm::count_if (NFC)
---
clang/lib/For
nico wrote:
Looks like this breaks tests on macOS:
http://45.33.8.238/macm1/107398/step_6.txt
Please take a look and revert for now if it takes a while to fix.
https://github.com/llvm/llvm-project/pull/140874
___
cfe-commits mailing list
cfe-commits@
https://github.com/DanielCChen edited
https://github.com/llvm/llvm-project/pull/141439
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -46,20 +46,20 @@ TEST(ClangTidyOptionsProvider, InMemoryFileSystems) {
FileOptionsProvider FileOpt({}, {}, {}, FileSystem);
- ClangTidyOptions File1Options =
+ llvm::ErrorOr File1Options =
FileOpt.getOptions("ProjectRoot/SubDir1/File.cpp");
- ClangTidyOptions F
@@ -0,0 +1,357 @@
+//===- unittests/StaticAnalyzer/BlockEntranceCallbackTest.cpp
-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/NagyDonat approved this pull request.
I also read the tests and the change LGTM if you move the explanations from the
very helpful review comments
https://github.com/llvm/llvm-project/pull/140924/files#r2107471659 and
https://github.com/llvm/llvm-project/pull/140924/files#r2
NagyDonat wrote:
Oops, forgot to update the title before merging, so it still refers to
`getDebugName` :sweat:
https://github.com/llvm/llvm-project/pull/141511
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
1 - 100 of 277 matches
Mail list logo
