[clang] [analyzer] Fix zext assertion failure in loop unrolling (PR #121203)

2024-12-28 Thread Balazs Benics via cfe-commits
https://github.com/steakhal approved this pull request. Looks wonderful now. I had recommended one simplification, but other than that we can merge this. Thanks again! https://github.com/llvm/llvm-project/pull/121203 ___ cfe-commits mailing list cfe-c

[clang] [analyzer] Fix zext assertion failure in loop unrolling (PR #121203)

2024-12-28 Thread Balazs Benics via cfe-commits
https://github.com/steakhal edited https://github.com/llvm/llvm-project/pull/121203 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Fix zext assertion failure in loop unrolling (PR #121203)

2024-12-28 Thread Balazs Benics via cfe-commits
@@ -283,10 +283,12 @@ static bool shouldCompletelyUnroll(const Stmt *LoopStmt, ASTContext &ASTCtx, llvm::APInt InitNum = Matches[0].getNodeAs("initNum")->getValue(); auto CondOp = Matches[0].getNodeAs("conditionOperator"); - if (InitNum.getBitWidth() != BoundNum.get

[clang] [analyzer] Fix zext assertion failure in loop unrolling (PR #121203)

2024-12-28 Thread Balazs Benics via cfe-commits
@@ -283,10 +283,12 @@ static bool shouldCompletelyUnroll(const Stmt *LoopStmt, ASTContext &ASTCtx, llvm::APInt InitNum = Matches[0].getNodeAs("initNum")->getValue(); auto CondOp = Matches[0].getNodeAs("conditionOperator"); - if (InitNum.getBitWidth() != BoundNum.get

[clang] [Clang] Implement P2280R4 Using unknown pointers and references in constant expressions (PR #95474)

2024-12-28 Thread via cfe-commits
@@ -1961,7 +1961,8 @@ namespace ConstexprConstructorRecovery { namespace Lifetime { void f() { -constexpr int &n = n; // expected-error {{constant expression}} expected-note {{use of reference outside its lifetime}} expected-warning {{not yet bound to a value}} +co

[clang] [Clang][P1061] Fix template arguments in local classes (PR #121225)

2024-12-28 Thread via cfe-commits
@@ -0,0 +1,14 @@ +// RUN: %clang_cc1 -fsyntax-only %s -verify +// expected-no-diagnostics + +template +int g() { + return [] (auto) -> int { +struct L { + int m = i; +}; +return 0; + } (42); cor3ntin wrote: That seems about right. But the fix

[clang] Delay marking pending incomplete decl chains until the end of `finishPendingActions`. (PR #121245)

2024-12-28 Thread via cfe-commits
cor3ntin wrote: Did you try creduce/cvise (although using that with modules might be a bit challenging) It would be really nice to have a repro so that we have a test case https://github.com/llvm/llvm-project/pull/121245 ___ cfe-commits mailing list c

[clang] [analyzer] Fix zext assertion failure in loop unrolling (PR #121203)

2024-12-28 Thread via cfe-commits
https://github.com/shenjunjiekoda updated https://github.com/llvm/llvm-project/pull/121203 >From 2cee5fc2abd0cf2979c9ba975906e659adcd4463 Mon Sep 17 00:00:00 2001 From: shenjunjie Date: Fri, 27 Dec 2024 14:08:55 + Subject: [PATCH 1/7] [analyzer] Fix zext assertion failure in loop unrolling

[clang] [analyzer] Fix zext assertion failure in loop unrolling (PR #121203)

2024-12-28 Thread via cfe-commits
shenjunjiekoda wrote: > Looks wonderful now. I had recommended one simplification, but other than > that we can merge this. > > Thanks again! Thank you for your patient and careful response! https://github.com/llvm/llvm-project/pull/121203 ___ cfe-c

[clang] [clang-format] Add `AllowShortNamespacesOnASingleLine` option (PR #105597)

2024-12-28 Thread Owen Pan via cfe-commits
@@ -361,9 +361,19 @@ class LineJoiner { const auto *FirstNonComment = TheLine->getFirstNonComment(); if (!FirstNonComment) return 0; + // FIXME: There are probably cases where we should use FirstNonComment // instead of TheLine->First. +if (TheLine

[clang] [clang-format] Add `AllowShortNamespacesOnASingleLine` option (PR #105597)

2024-12-28 Thread Owen Pan via cfe-commits
https://github.com/owenca approved this pull request. https://github.com/llvm/llvm-project/pull/105597 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [tysan] Convert TySan from function+module pass to just module pass (PR #120667)

2024-12-28 Thread Florian Hahn via cfe-commits
https://github.com/fhahn approved this pull request. LGTM, thanks! https://github.com/llvm/llvm-project/pull/120667 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-s390x-linux-lnt` running on `systemz-1` while building `clang-tools-extra` at step 7 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/136/builds/2164 Here is the relevant piece

[clang] 8e965d8 - [analyzer] Fix zext assertion failure in loop unrolling (#121203)

2024-12-28 Thread via cfe-commits
Author: JOSTAR Date: 2024-12-28T11:09:29+01:00 New Revision: 8e965d89c9624c184c48806dc39d50209265f0f8 URL: https://github.com/llvm/llvm-project/commit/8e965d89c9624c184c48806dc39d50209265f0f8 DIFF: https://github.com/llvm/llvm-project/commit/8e965d89c9624c184c48806dc39d50209265f0f8.diff LOG: [

[clang] [analyzer] Fix zext assertion failure in loop unrolling (PR #121203)

2024-12-28 Thread Balazs Benics via cfe-commits
https://github.com/steakhal closed https://github.com/llvm/llvm-project/pull/121203 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Add -fuse-lipo option (PR #121231)

2024-12-28 Thread Ashley Hauck via cfe-commits
https://github.com/khyperia updated https://github.com/llvm/llvm-project/pull/121231 >From 33b542152876b9ccbf42cc3d070d582c32145477 Mon Sep 17 00:00:00 2001 From: khyperia Date: Fri, 27 Dec 2024 23:03:58 +0100 Subject: [PATCH 1/2] Add -fuse-lipo option --- clang/include/clang/Driver/Options.t

[clang] Add -fuse-lipo option (PR #121231)

2024-12-28 Thread Ashley Hauck via cfe-commits
khyperia wrote: @carlocab I've removed the temporary std::string and added a guess at basic tests. Let me know if there's additional tests in particular that you're thinking of! https://github.com/llvm/llvm-project/pull/121231 ___ cfe-commits mailing

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64le-rhel` running on `ppc64le-clang-rhel-test` while building `clang-tools-extra` at step 7 "test-build-unified-tree-check-all". Full details are available at: https://lab.llvm.org/buildbot/#/builders/145/builds/409

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-s390x-linux` running on `systemz-1` while building `clang-tools-extra` at step 5 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/42/builds/2535 Here is the relevant piece of the

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread via cfe-commits
4m4n-x-B4w4ne wrote: > Excuse me, I've reverted this. Can you help on what thing , It went wrong and like ,What shall I need to do to make it done? https://github.com/llvm/llvm-project/pull/120087 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread NAKAMURA Takumi via cfe-commits
chapuni wrote: Excuse me, I've reverted this. https://github.com/llvm/llvm-project/pull/120087 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Add `AllowShortNamespacesOnASingleLine` option (PR #105597)

2024-12-28 Thread Owen Pan via cfe-commits
https://github.com/owenca edited https://github.com/llvm/llvm-project/pull/105597 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-aarch64-sve2-vla-2stage` running on `linaro-g4-01` while building `clang-tools-extra` at step 12 "ninja check 2". Full details are available at: https://lab.llvm.org/buildbot/#/builders/199/builds/482 Here is the rele

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64le-linux-multistage` running on `ppc64le-clang-multistage-test` while building `clang-tools-extra` at step 5 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/76/builds/5676

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-aarch64-global-isel` running on `linaro-clang-aarch64-global-isel` while building `clang-tools-extra` at step 7 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/125/builds/4518

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-armv7-global-isel` running on `linaro-clang-armv7-global-isel` while building `clang-tools-extra` at step 7 "ninja check 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/39/builds/3628 Here

[clang] Add -fuse-lipo option (PR #121231)

2024-12-28 Thread Carlo Cabrera via cfe-commits
https://github.com/carlocab edited https://github.com/llvm/llvm-project/pull/121231 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Add -fuse-lipo option (PR #121231)

2024-12-28 Thread Carlo Cabrera via cfe-commits
https://github.com/carlocab commented: Thanks for the PR! This change needs new tests to be added. https://github.com/llvm/llvm-project/pull/121231 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/

[clang] Add -fuse-lipo option (PR #121231)

2024-12-28 Thread Carlo Cabrera via cfe-commits
@@ -910,7 +910,10 @@ void darwin::Lipo::ConstructJob(Compilation &C, const JobAction &JA, CmdArgs.push_back(II.getFilename()); } - const char *Exec = Args.MakeArgString(getToolChain().GetProgramPath("lipo")); + std::string LipoName = + std::string(Args.getLastArg

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `lldb-remote-linux-ubuntu` running on `as-builder-9` while building `clang` at step 16 "test-check-lldb-api". Full details are available at: https://lab.llvm.org/buildbot/#/builders/195/builds/2861 Here is the relevant piece

[clang-tools-extra] [clang-tidy] bugprone-unhandled-self-assignment: fix smart pointer check against std::unique_ptr type (PR #121266)

2024-12-28 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang-tools-extra] [clang-tidy] bugprone-unhandled-self-assignment: fix smart pointer check against std::unique_ptr type (PR #121266)

2024-12-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tidy @llvm/pr-subscribers-clang-tools-extra Author: None (flovent) Changes Unlike other standard smart pointer types, std::unique_ptr has two template arguments. testcase need to be updated too. --- Full diff: https://github.com/llvm/llvm-project

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `ppc64le-lld-multistage-test` running on `ppc64le-lld-multistage-test` while building `clang` at step 12 "build-stage2-unified-tree". Full details are available at: https://lab.llvm.org/buildbot/#/builders/168/builds/6991 He

[clang-tools-extra] [clang-tidy][clangd] Fixed removeFunctionArgs don't remove comma for use-ranges check (PR #118568)

2024-12-28 Thread Julian Schmidt via cfe-commits
@@ -164,6 +164,33 @@ void UseRangesCheck::registerMatchers(MatchFinder *Finder) { static void removeFunctionArgs(DiagnosticBuilder &Diag, const CallExpr &Call, ArrayRef Indexes, const ASTContext &Ctx) { + auto GetC

[clang-tools-extra] [clang-tidy] swap cppcoreguidelines-narrowing-conversions and bugprone-narrowing-conversions (PR #120245)

2024-12-28 Thread Julian Schmidt via cfe-commits
https://github.com/5chmidti approved this pull request. LGTM, thanks https://github.com/llvm/llvm-project/pull/120245 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Mention std::forward_list in container-size-empty doc (PR #120701)

2024-12-28 Thread Julian Schmidt via cfe-commits
https://github.com/5chmidti approved this pull request. https://github.com/llvm/llvm-project/pull/120701 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [RISCV] Add Qualcomm uC Xqciac (Load-Store Adress calculation) extension (PR #121037)

2024-12-28 Thread Craig Topper via cfe-commits
https://github.com/topperc approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/121037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Delay marking pending incomplete decl chains until the end of `finishPendingActions`. (PR #121245)

2024-12-28 Thread Michael Park via cfe-commits
mpark wrote: > Did you try creduce/cvise (although using that with modules might be a bit > challenging) It would be really nice to have a repro so that we have a test > case I did. It did help me reduce it significantly, but it still involves a few modules that are difficult to reduce. I sti

[clang] Reapply "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test (#121160)" (PR #121162)

2024-12-28 Thread Justin Bogner via cfe-commits
bogner wrote: These tests are failing for me locally on windows now. Why are you removing this? https://github.com/llvm/llvm-project/pull/121162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cf

[clang] Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test" again (PR #121280)

2024-12-28 Thread Justin Bogner via cfe-commits
https://github.com/bogner created https://github.com/llvm/llvm-project/pull/121280 This reverts #121162, which was a reapply of the previous revert earlier in #121160 - The change blatantly breaks tests on windows and it isn't clear why it's being made. This reverts commit 8e9fda1c1140e067c53

[clang] Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test" again (PR #121280)

2024-12-28 Thread Justin Bogner via cfe-commits
https://github.com/bogner updated https://github.com/llvm/llvm-project/pull/121280 >From 650dd5a50a6228124dc40676ea08a963ecdf3bb0 Mon Sep 17 00:00:00 2001 From: Justin Bogner Date: Sat, 28 Dec 2024 21:35:03 -0700 Subject: [PATCH 1/2] Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test

[clang] Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test" again (PR #121280)

2024-12-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Justin Bogner (bogner) Changes This reverts #121162, which was a reapply of the previous revert earlier in #121160 - The change blatantly breaks tests on windows and it isn't clear why it's being made. This reverts commit 8e9fda1c1140e06

[clang] 02e8972 - Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test" again (#121280)

2024-12-28 Thread via cfe-commits
Author: Justin Bogner Date: 2024-12-28T21:48:25-07:00 New Revision: 02e8972c378bf60cc16a85815d29faafdbad7180 URL: https://github.com/llvm/llvm-project/commit/02e8972c378bf60cc16a85815d29faafdbad7180 DIFF: https://github.com/llvm/llvm-project/commit/02e8972c378bf60cc16a85815d29faafdbad7180.diff

[clang] Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test" again (PR #121280)

2024-12-28 Thread Justin Bogner via cfe-commits
https://github.com/bogner closed https://github.com/llvm/llvm-project/pull/121280 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Reapply "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test (#121160)" (PR #121162)

2024-12-28 Thread Justin Bogner via cfe-commits
bogner wrote: I've gone ahead and reverted it. https://github.com/llvm/llvm-project/pull/121162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test" again (PR #121280)

2024-12-28 Thread Justin Bogner via cfe-commits
https://github.com/bogner edited https://github.com/llvm/llvm-project/pull/121280 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Absoultify paths in dependency file output (PR #117458)

2024-12-28 Thread via cfe-commits
xtexChooser wrote: cc @MaskRay https://github.com/llvm/llvm-project/pull/117458 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 3496e96 - [clang][bytecode] Add a missing break

2024-12-28 Thread Timm Bäder via cfe-commits
Author: Timm Bäder Date: 2024-12-28T14:17:06+01:00 New Revision: 3496e96f78c46f5b94c1892f97c470fd89293795 URL: https://github.com/llvm/llvm-project/commit/3496e96f78c46f5b94c1892f97c470fd89293795 DIFF: https://github.com/llvm/llvm-project/commit/3496e96f78c46f5b94c1892f97c470fd89293795.diff LO

[clang-tools-extra] [clang-tidy] add depercation warning for non-whitelisted global options (PR #121057)

2024-12-28 Thread Julian Schmidt via cfe-commits
5chmidti wrote: > Good point, maybe we can use more smart way to do warning: only warn the > global option is set but local option is not. > > Have some redundant options is not the aim for this deprecation things, we > only want to avoid user suddenly find the behaviors of lots of check are

[clang] [llvm] [RISCV] Add Qualcomm uC Xqciac (Load-Store Adress calculation) extension (PR #121037)

2024-12-28 Thread Sam Elliott via cfe-commits
@@ -184,6 +191,37 @@ let hasSideEffects = 0, mayLoad = 0, mayStore = 0 in { } // hasSideEffects = 0, mayLoad = 0, mayStore = 0 } // Predicates = [HasVendorXqcia, IsRV32], DecoderNamespace = "Xqcia" +let Predicates = [HasVendorXqciac, IsRV32], DecoderNamespace = "Xqciac" in { +

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Julian Schmidt via cfe-commits
@@ -218,6 +218,7 @@ RegistryMaps::RegistryMaps() { REGISTER_MATCHER(cxxTryStmt); REGISTER_MATCHER(cxxUnresolvedConstructExpr); REGISTER_MATCHER(decayedType); + REGISTER_MATCHER(dependentNameType); 5chmidti wrote: Please sort this in alphabetically http

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Julian Schmidt via cfe-commits
@@ -7711,6 +7711,16 @@ AST_MATCHER_P(DecayedType, hasDecayedType, internal::Matcher, return InnerType.matches(Node.getDecayedType(), Finder, Builder); } +/// Matches dependent name type 5chmidti wrote: nit: `a dependent name type` (+1 in release notes) ht

[clang-tools-extra] [clang-tidy] bugprone-unhandled-self-assignment: fix smart pointer check against std::unique_ptr type (PR #121266)

2024-12-28 Thread via cfe-commits
https://github.com/flovent created https://github.com/llvm/llvm-project/pull/121266 Unlike other standard smart pointer types, std::unique_ptr has two template arguments. testcase need to be updated too. >From 009178ad073cd76630418e74092907e1d9dc0d85 Mon Sep 17 00:00:00 2001 From: flovent Dat

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Amr Hesham via cfe-commits
https://github.com/AmrDeveloper updated https://github.com/llvm/llvm-project/pull/121263 >From bdbd11db849506c4d99036dd674f03d1eda815cc Mon Sep 17 00:00:00 2001 From: AmrDeveloper Date: Sat, 28 Dec 2024 13:23:33 +0100 Subject: [PATCH 1/2] [Clang][ASTMatcher] Add `dependentNameType` Matcher ---

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Amr Hesham via cfe-commits
@@ -218,6 +218,7 @@ RegistryMaps::RegistryMaps() { REGISTER_MATCHER(cxxTryStmt); REGISTER_MATCHER(cxxUnresolvedConstructExpr); REGISTER_MATCHER(decayedType); + REGISTER_MATCHER(dependentNameType); AmrDeveloper wrote: Done, thank you https://github.com/

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Amr Hesham via cfe-commits
@@ -7711,6 +7711,16 @@ AST_MATCHER_P(DecayedType, hasDecayedType, internal::Matcher, return InnerType.matches(Node.getDecayedType(), Finder, Builder); } +/// Matches dependent name type AmrDeveloper wrote: Done https://github.com/llvm/llvm-project/pull/12

[clang] Reapply "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test (#121160)" (PR #121162)

2024-12-28 Thread Carlo Cabrera via cfe-commits
carlocab wrote: I don't really understand why this is needed either. Should we revert? https://github.com/llvm/llvm-project/pull/121162 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Revert "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test" again (PR #121280)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-cmake-x86_64-avx512-win` running on `avx512-intel64-win` while building `clang` at step 4 "cmake stage 1". Full details are available at: https://lab.llvm.org/buildbot/#/builders/81/builds/3419 Here is the relevant pi

[clang] Reapply "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test (#121160)" (PR #121162)

2024-12-28 Thread Fangrui Song via cfe-commits
MaskRay wrote: This is not correct. Windows GetLinkerPath does append `.exe`. For certain tests where Windows's .exe and backslashes are annoying, we could employ `UNSUPPORTED: system-windows`. I am puzzled by the description about Android. https://github.com/llvm/llvm-project/pull/121162

[clang] Reapply "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test (#121160)" (PR #121162)

2024-12-28 Thread Vitaly Buka via cfe-commits
vitalybuka wrote: > I've gone ahead and reverted it. Thanks for reverting. Could you please provide some information on how it fails? Error output would be enough. Upstream windows bot are OK. >Windows GetLinkerPath does append .exe How upstream bots pass? E.g. the one above is unrelated flak

[clang] 1557eed - [RISCV] Add Qualcomm uC Xqciac (Load-Store Adress calculation) extension (#121037)

2024-12-28 Thread via cfe-commits
Author: quic_hchandel Date: 2024-12-29T11:14:12+05:30 New Revision: 1557eeda738d7dbe51d2f52fce28a1fd6f5844ce URL: https://github.com/llvm/llvm-project/commit/1557eeda738d7dbe51d2f52fce28a1fd6f5844ce DIFF: https://github.com/llvm/llvm-project/commit/1557eeda738d7dbe51d2f52fce28a1fd6f5844ce.diff

[clang] [llvm] [RISCV] Add Qualcomm uC Xqciac (Load-Store Adress calculation) extension (PR #121037)

2024-12-28 Thread Sudharsan Veeravalli via cfe-commits
https://github.com/svs-quic closed https://github.com/llvm/llvm-project/pull/121037 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Reapply "[nfc][Driver] Remove {{(.exe)?}} from sanitizer test (#121160)" (PR #121162)

2024-12-28 Thread Vitaly Buka via cfe-commits
vitalybuka wrote: > I am puzzled by the description about Android. There is this configured buildkite https://buildkite.com/llvm-project/github-pull-requests/builds/132244 It's Windows, but only Android lines of the test fail. https://github.com/llvm/llvm-project/pull/121162 ___

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Nathan Ridge via cfe-commits
@@ -2536,6 +2536,15 @@ Node Matchers matches "decltype(i + j)" +MatcherStmt>dependentNameTypeMatcherDependentNameType>... HighCo

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Nathan Ridge via cfe-commits
@@ -1912,6 +1912,21 @@ TEST_P(ASTMatchersTest, DeducedTemplateSpecializationType) { deducedTemplateSpecializationType())); } +TEST_P(ASTMatchersTest, DependentNameType) { + if (!GetParam().isCXX()) { +// FIXME: Add a test for `dependentNameType()` that does

[clang] [clang-format] Add `AllowShortNamespacesOnASingleLine` option (PR #105597)

2024-12-28 Thread Galen Elias via cfe-commits
https://github.com/galenelias updated https://github.com/llvm/llvm-project/pull/105597 >From 93eb3d89652607173f4f68fce7dcc5b2bd33f266 Mon Sep 17 00:00:00 2001 From: Galen Elias Date: Wed, 21 Aug 2024 16:33:42 -0700 Subject: [PATCH 01/15] clang-format: Add "AllowShortNamespacesOnASingleLine" op

[clang] 8e32959 - [CIR] Upstream initial attribute support (#121069)

2024-12-28 Thread via cfe-commits
Author: David Olsen Date: 2024-12-28T14:02:15-08:00 New Revision: 8e329593313bb792592529ee825a52683108df99 URL: https://github.com/llvm/llvm-project/commit/8e329593313bb792592529ee825a52683108df99 DIFF: https://github.com/llvm/llvm-project/commit/8e329593313bb792592529ee825a52683108df99.diff L

[clang] [CIR] Upstream initial attribute support (PR #121069)

2024-12-28 Thread David Olsen via cfe-commits
https://github.com/dkolsen-pgi closed https://github.com/llvm/llvm-project/pull/121069 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] 537d4e9 - Revert "Added options to readability-implicit-bool-conversion (#120087)"

2024-12-28 Thread NAKAMURA Takumi via cfe-commits
Author: NAKAMURA Takumi Date: 2024-12-28T17:47:00+09:00 New Revision: 537d4e9d21be1f5e40a780f570663b04572765af URL: https://github.com/llvm/llvm-project/commit/537d4e9d21be1f5e40a780f570663b04572765af DIFF: https://github.com/llvm/llvm-project/commit/537d4e9d21be1f5e40a780f570663b04572765af.dif

[clang] 811e1f4 - clang/test/CoverageMapping/single-byte-counters.cpp: Align to the final form to fill linefeeds.

2024-12-28 Thread NAKAMURA Takumi via cfe-commits
Author: NAKAMURA Takumi Date: 2024-12-28T17:47:09+09:00 New Revision: 811e1f4661bca4a2b5c93d30f54c3aa338f175e9 URL: https://github.com/llvm/llvm-project/commit/811e1f4661bca4a2b5c93d30f54c3aa338f175e9 DIFF: https://github.com/llvm/llvm-project/commit/811e1f4661bca4a2b5c93d30f54c3aa338f175e9.dif

[clang] Add -fuse-lipo option (PR #121231)

2024-12-28 Thread Ashley Hauck via cfe-commits
@@ -910,7 +910,10 @@ void darwin::Lipo::ConstructJob(Compilation &C, const JobAction &JA, CmdArgs.push_back(II.getFilename()); } - const char *Exec = Args.MakeArgString(getToolChain().GetProgramPath("lipo")); + std::string LipoName = + std::string(Args.getLastArg

[clang-tools-extra] Added options to readability-implicit-bool-conversion (PR #120087)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64le-linux-test-suite` running on `ppc64le-clang-test-suite` while building `clang-tools-extra` at step 6 "test-build-unified-tree-check-all". Full details are available at: https://lab.llvm.org/buildbot/#/builders/9

[clang] [Clang][P1061] Fix template arguments in local classes (PR #121225)

2024-12-28 Thread Younan Zhang via cfe-commits
@@ -0,0 +1,14 @@ +// RUN: %clang_cc1 -fsyntax-only %s -verify +// expected-no-diagnostics + +template +int g() { + return [] (auto) -> int { +struct L { + int m = i; +}; +return 0; + } (42); zyn0217 wrote: Yeah, probably also worth an assertio

[clang] [lld] [llvm] [clang][MIPS] Add support for mipsel-windows-* targets (PR #107744)

2024-12-28 Thread Hervé Poussineau via cfe-commits
hpoussin wrote: Current list of PRs to have a usable toolchain (except linker): - #120876 [MC][Mips] Generate required IMAGE_REL_MIPS_PAIR relocation - #120877 [MC][CodeGen][Mips] Add CodeView mapping - #120912 [Mips] Handle declspec(dllimport) on mipsel-windows-* triples - #121041 [Clang][MIPS]

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Amr Hesham (AmrDeveloper) Changes Add AST Matcher for `DependentNameType` Fixes: https://github.com/llvm/llvm-project/issues/121240 --- Full diff: https://github.com/llvm/llvm-project/pull/121263.diff 7 Files Affected: - (modified) cla

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Amr Hesham via cfe-commits
https://github.com/AmrDeveloper created https://github.com/llvm/llvm-project/pull/121263 Add AST Matcher for `DependentNameType` Fixes: https://github.com/llvm/llvm-project/issues/121240 >From 21873d01238f8da0eb0f19656801347400afae65 Mon Sep 17 00:00:00 2001 From: AmrDeveloper Date: Sat, 28 D

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread Amr Hesham via cfe-commits
https://github.com/AmrDeveloper updated https://github.com/llvm/llvm-project/pull/121263 >From bdbd11db849506c4d99036dd674f03d1eda815cc Mon Sep 17 00:00:00 2001 From: AmrDeveloper Date: Sat, 28 Dec 2024 13:23:33 +0100 Subject: [PATCH] [Clang][ASTMatcher] Add `dependentNameType` Matcher --- cl

[clang] [Clang][ASTMatcher] Add `dependentNameType` Matcher (PR #121263)

2024-12-28 Thread via cfe-commits
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 48bf0a9457fd60d0872d9b9b4804a95c833a72e1 21873d01238f8da0eb0f19656801347400afae65 --e

[clang] e86b68f - [clang][bytecode] Add support for typeid pointers (#121251)

2024-12-28 Thread via cfe-commits
Author: Timm Baeder Date: 2024-12-28T14:07:01+01:00 New Revision: e86b68ff560aaf5fc723eaa8d8418892b2456e12 URL: https://github.com/llvm/llvm-project/commit/e86b68ff560aaf5fc723eaa8d8418892b2456e12 DIFF: https://github.com/llvm/llvm-project/commit/e86b68ff560aaf5fc723eaa8d8418892b2456e12.diff L

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/121251 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64le-rhel` running on `ppc64le-clang-rhel-test` while building `clang` at step 5 "build-unified-tree". Full details are available at: https://lab.llvm.org/buildbot/#/builders/145/builds/4101 Here is the relevant pi

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `clang-ppc64-aix` running on `aix-ppc64` while building `clang` at step 5 "build-unified-tree". Full details are available at: https://lab.llvm.org/buildbot/#/builders/64/builds/1803 Here is the relevant piece of the build lo

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-ppc64le-linux` running on `ppc64le-sanitizer` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/72/builds/6693 Here is the relevant piece of the bu

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via 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/5214 Here is the releva

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-aarch64-linux` running on `sanitizer-buildbot8` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/51/builds/8284 Here is the relevant piece of the

[clang] [clang][bytecode] Add support for typeid pointers (PR #121251)

2024-12-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux` running on `sanitizer-buildbot2` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/66/builds/8030 Here is the relevant piece of the b