[clang-tools-extra] [clangd] Add CodePatterns config option under Completion (PR #137613)

2025-05-04 Thread Nathan Ridge via cfe-commits
HighCommander4 wrote: > There is 2 different features involved here. > > [...] > > In the feature i'm trying to implement, number 1 should be OFF if > CodePatterns is set to None but number 2 should still be working. That makes sense to me, as a user expectation. The code I linked to in my pr

[clang-tools-extra] [clangd] Add CodePatterns config option under Completion (PR #137613)

2025-05-04 Thread via cfe-commits
Noustaa wrote: There is 2 different features involved here. 1) The fact that if i type "inc" it will suggest me both includes snippets Like the below image the ones tagged LSP on the right side ![image](https://github.com/user-attachments/assets/a21e1db2-f057-4460-921e-d62d25efb794) 2) The fact

[clang-tools-extra] [clangd] Store documentation when indexing standard library (PR #133681)

2025-05-04 Thread Nathan Ridge via cfe-commits
@@ -158,6 +159,42 @@ TEST(StdLibTests, EndToEnd) { UnorderedElementsAre(StdlibSymbol("list"), StdlibSymbol("vector"))); } +TEST(StdLibTests, StdLibDocComments) { + Config Cfg; + Cfg.Index.StandardLibrary = true; + WithContextValue Enabled(Config::Key, std::move(Cfg));

[clang-tools-extra] [clangd] Store documentation when indexing standard library (PR #133681)

2025-05-04 Thread Nathan Ridge via cfe-commits
https://github.com/HighCommander4 updated https://github.com/llvm/llvm-project/pull/133681 >From ce15317543b135b7aa86d4987fb0c25abd597d56 Mon Sep 17 00:00:00 2001 From: Nathan Ridge Date: Mon, 31 Mar 2025 02:25:45 -0400 Subject: [PATCH] [clangd] Store documentation when indexing standard librar

[clang-tools-extra] [clangd] Store documentation when indexing standard library (PR #133681)

2025-05-04 Thread kadir çetinkaya via cfe-commits
https://github.com/kadircet approved this pull request. thanks, LGTM! https://github.com/llvm/llvm-project/pull/133681 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clangd] Store documentation when indexing standard library (PR #133681)

2025-05-04 Thread kadir çetinkaya via cfe-commits
@@ -158,6 +159,42 @@ TEST(StdLibTests, EndToEnd) { UnorderedElementsAre(StdlibSymbol("list"), StdlibSymbol("vector"))); } +TEST(StdLibTests, StdLibDocComments) { + Config Cfg; + Cfg.Index.StandardLibrary = true; + WithContextValue Enabled(Config::Key, std::move(Cfg));

[clang-tools-extra] [clangd] Store documentation when indexing standard library (PR #133681)

2025-05-04 Thread kadir çetinkaya via cfe-commits
@@ -158,6 +158,43 @@ TEST(StdLibTests, EndToEnd) { UnorderedElementsAre(StdlibSymbol("list"), StdlibSymbol("vector"))); } +TEST(StdLibTests, StdLibDocComments) { + Config Cfg; + Cfg.Index.StandardLibrary = true; + WithContextValue Enabled(Config::Key, std::move(Cfg));

[clang-tools-extra] [clangd] Store documentation when indexing standard library (PR #133681)

2025-05-04 Thread kadir çetinkaya via cfe-commits
https://github.com/kadircet edited https://github.com/llvm/llvm-project/pull/133681 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][ptrauth] add support for options parameter to __ptrauth (PR #136828)

2025-05-04 Thread Oliver Hunt via cfe-commits
@@ -8374,20 +8376,212 @@ static void HandlePtrAuthQualifier(ASTContext &Ctx, QualType &T, IsInvalid |= !S.checkPointerAuthDiscriminatorArg( ExtraDiscriminatorArg, Sema::PADAK_ExtraDiscPtrAuth, ExtraDiscriminator); - if (IsInvalid) { -Attr.setInvalid(); -retur

[clang] [C23] Implement WG14 N3037 (PR #132939)

2025-05-04 Thread Martin Uecker via cfe-commits
uecker wrote: So one usability issue I noticed that we also had in GCC is that the warning about declaring structs in function prototypes (-Wvisibility) does not make sense anymore and becomes annoying. We turned it off in C23 for complete (but not for incomplete) structs) without tag. http

[clang] [llvm] [clang] [OpenMP] New OpenMP 6.0 self_maps clause - CodeGen (PR #134131)

2025-05-04 Thread via cfe-commits
@@ -236,6 +236,8 @@ enum class OpenMPOffloadMappingFlags : uint64_t { // dynamic. // This is an OpenMP extension for the sake of OpenACC support. OMP_MAP_OMPX_HOLD = 0x2000, + /// Self directs mapping without creating a separate device copy. + OMP_MAP_SELF = 0x4000, ---

[clang] [llvm] [clang][OpenMP] New OpenMP 6.0 threadset clause (PR #135807)

2025-05-04 Thread via cfe-commits
Ritanya-B-Bharadwaj wrote: @alexey-bataev I've addressed your comments. For anything that's out of scope for this patch, I'll handle it in upcoming patches. Let me know if this can be merged. https://github.com/llvm/llvm-project/pull/135807 ___ cfe-c

[clang] [clang-linker-wrapper] Remove unused local variables (NFC) (PR #138480)

2025-05-04 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/8760 Here is the releva

[clang] [llvm] [BasicAA] Do not decompose past casts with different index width (PR #119365)

2025-05-04 Thread via cfe-commits
liushuyu wrote: Hi there, It seems like after this pull request, the following C code will crash LLVM if LLVM assertion is enabled: Godbolt link: https://godbolt.org/z/91bqnYsb4 ```c #include #include #if SIZE_MAX != UINT32_MAX #error "This reproducer only works on 32-bit targets" #endif

[clang] [clang][ptrauth] Add support for querying the ptrauth schema of a type (PR #138482)

2025-05-04 Thread Oliver Hunt via cfe-commits
ojhunt wrote: Also the associated tests do not pass yet as they're blocked on https://github.com/llvm/llvm-project/pull/136828 and other ptrauth PRs https://github.com/llvm/llvm-project/pull/138482 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

[clang] [clang][ptrauth] Add support for querying the ptrauth schema of a type (PR #138482)

2025-05-04 Thread Oliver Hunt via cfe-commits
ojhunt wrote: Adding @cor3ntin and @AaronBallman in case they have a better idea for getting the function key to Sema than having it duplicated in langopts and codegen options https://github.com/llvm/llvm-project/pull/138482 ___ cfe-commits mailing l

[clang] [clang][ptrauth] Add support for querying the ptrauth schema of a type (PR #138482)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang @llvm/pr-subscribers-clang-codegen Author: Oliver Hunt (ojhunt) Changes This adds a number of builtins to query the ptrauth schema of an arbitrary type in a way that can be fed into other ptrauth qualifiers. --- Patch is 34.90 KiB, truncated to

[clang] [clang][ptrauth] Add support for querying the ptrauth schema of a type (PR #138482)

2025-05-04 Thread Oliver Hunt via cfe-commits
https://github.com/ojhunt created https://github.com/llvm/llvm-project/pull/138482 This adds a number of builtins to query the ptrauth schema of an arbitrary type in a way that can be fed into other ptrauth qualifiers. >From e84310f8a57a3f9b75dfc7260137e984d7f55f87 Mon Sep 17 00:00:00 2001 Fro

[clang] [llvm] [OpenMP] Remove 'libomptarget.devicertl.a' fatbinary and use static library (PR #126143)

2025-05-04 Thread Ye Luo via cfe-commits
@@ -142,99 +125,40 @@ function(compileDeviceRTLLibrary target_name target_triple) endif() set_property(DIRECTORY APPEND PROPERTY ADDITIONAL_MAKE_CLEAN_FILES ${outfile}) -list(APPEND bc_files ${outfile}) +list(APPEND obj_files ${CMAKE_CURRENT_BINARY_DIR}/${out

[clang] [Clang][OpenCL][AMDGPU] OpenCL Kernel stubs should be assigned alwaysinline attribute (PR #137769)

2025-05-04 Thread Aniket Lal via cfe-commits
https://github.com/lalaniket8 updated https://github.com/llvm/llvm-project/pull/137769 >From aa151fcda1cb73edc9564f6e2494699046254350 Mon Sep 17 00:00:00 2001 From: anikelal Date: Tue, 29 Apr 2025 13:59:48 +0530 Subject: [PATCH] add alwaysinline attribute to stubs --- clang/lib/CodeGen/CodeGe

[clang] [clang-linker-wrapper] Remove unused local variables (NFC) (PR #138480)

2025-05-04 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `openmp-s390x-linux` running on `systemz-1` while building `clang` at step 6 "test-openmp". Full details are available at: https://lab.llvm.org/buildbot/#/builders/88/builds/11249 Here is the relevant piece of the build log f

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Kees Cook via cfe-commits
@@ -34,6 +34,7 @@ class SanitizerArgs { std::vector CoverageIgnorelistFiles; std::vector BinaryMetadataIgnorelistFiles; int CoverageFeatures = 0; + int StackDepthCallbackMin = 0; kees wrote: Fixed. https://github.com/llvm/llvm-project/pull/138323 _

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Kees Cook via cfe-commits
https://github.com/kees updated https://github.com/llvm/llvm-project/pull/138323 >From 027ff82e738f6bdf82f958a4774eb05d321b23ef Mon Sep 17 00:00:00 2001 From: Kees Cook Date: Fri, 2 May 2025 10:51:02 -0700 Subject: [PATCH] [sancov] Introduce optional callback for stack-depth tracking Normally -

[clang] [clang-linker-wrapper] Remove unused local variables (NFC) (PR #138480)

2025-05-04 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata closed https://github.com/llvm/llvm-project/pull/138480 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] aafbdde - [clang-linker-wrapper] Remove unused local variables (NFC) (#138480)

2025-05-04 Thread via cfe-commits
Author: Kazu Hirata Date: 2025-05-04T20:36:07-07:00 New Revision: aafbddee808b8c81e2401ddcb6803f582ccb6892 URL: https://github.com/llvm/llvm-project/commit/aafbddee808b8c81e2401ddcb6803f582ccb6892 DIFF: https://github.com/llvm/llvm-project/commit/aafbddee808b8c81e2401ddcb6803f582ccb6892.diff L

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Kees Cook via cfe-commits
https://github.com/kees updated https://github.com/llvm/llvm-project/pull/138323 >From 729d249d3848c2e417174ed935bfefde5c048dc7 Mon Sep 17 00:00:00 2001 From: Kees Cook Date: Fri, 2 May 2025 10:51:02 -0700 Subject: [PATCH] [sancov] Introduce optional callback for stack-depth tracking Normally -

[clang] [clang] Warn about deprecated volatile-qualified return types (PR #137899)

2025-05-04 Thread via cfe-commits
halbi2 wrote: @shafik I have changed the commit message, can you please look again? https://github.com/llvm/llvm-project/pull/137899 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Warn about deprecated volatile-qualified return types (PR #137899)

2025-05-04 Thread via cfe-commits
https://github.com/halbi2 updated https://github.com/llvm/llvm-project/pull/137899 >From 238d73776c85ed22386ac494f275261638cd40d3 Mon Sep 17 00:00:00 2001 From: halbi2 Date: Mon, 28 Apr 2025 20:07:43 -0400 Subject: [PATCH] [clang] Warn about deprecated volatile-qualified return types The old c

[clang] [clang] Warn about deprecated volatile-qualified return types (PR #137899)

2025-05-04 Thread via cfe-commits
@@ -231,6 +231,13 @@ namespace DeprecatedVolatile { a = c = a; b += a; } + + volatile struct amber jurassic(); halbi2 wrote: Lines 207-215 are that, are they not? https://github.com/llvm/llvm-project/pull/137899

[clang] [clang] Warn about deprecated volatile-qualified return types (PR #137899)

2025-05-04 Thread via cfe-commits
@@ -5056,13 +5056,13 @@ static TypeSourceInfo *GetFullTypeForDeclarator(TypeProcessingState &state, S.Diag(DeclType.Loc, diag::err_func_returning_qualified_void) << T; } else diagnoseRedundantReturnTypeQualifiers(S, T, D, chunkIndex); - -//

[clang] [clang-linker-wrapper] Remove unused local variables (NFC) (PR #138480)

2025-05-04 Thread Joseph Huber via cfe-commits
https://github.com/jhuber6 approved this pull request. https://github.com/llvm/llvm-project/pull/138480 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-linker-wrapper] Remove unused local variables (NFC) (PR #138480)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/138480.diff 1 Files Affected: - (modified) clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp (-5) ``diff diff --git a/clang/

[clang] [clang-linker-wrapper] Remove unused local variables (NFC) (PR #138480)

2025-05-04 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/138480 None >From fb19c7799d3ac769eb75d9fee0a3a8b7827d1908 Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sun, 4 May 2025 19:13:57 -0700 Subject: [PATCH] [clang-linker-wrapper] Remove unused local variables (

[clang] [Clang][Cygwin] attempt to fix building shared libclang. (PR #138351)

2025-05-04 Thread via cfe-commits
jeremyd2019 wrote: Surprisingly, it looks like Cygwin (@tyan0) is working around the export limit on gcc by setting `-DCMAKE_SHARED_LINKER_FLAGS=-fvisibility=hidden`. I was surprised that worked on gcc - maybe it can be generalized here instead of fighting to build only static with gcc to the

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Kees Cook via cfe-commits
kees wrote: > This is also missing flag and IR tests. Oh, yes, I will add those. Thanks! https://github.com/llvm/llvm-project/pull/138323 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Kees Cook via cfe-commits
@@ -1078,22 +1092,44 @@ void ModuleSanitizerCoverage::InjectCoverageAtBlock(Function &F, BasicBlock &BB, Store->setNoSanitizeMetadata(); } if (Options.StackDepth && IsEntryBB && !IsLeafFunc) { -// Check stack depth. If it's the deepest so far, record it. Modu

[libunwind] [libunwind] [SEH] Set NonVolatileRegisters before calling a personality function (PR #137951)

2025-05-04 Thread Jacek Caban via cfe-commits
https://github.com/cjacek approved this pull request. Looks reasonable to me overall. https://github.com/llvm/llvm-project/pull/137951 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] [libunwind] [SEH] Set NonVolatileRegisters before calling a personality function (PR #137951)

2025-05-04 Thread Jacek Caban via cfe-commits
@@ -212,6 +238,21 @@ __libunwind_seh_personality(int version, _Unwind_Action state, ms_exc.ExceptionInformation[2] = state; DISPATCHER_CONTEXT *disp_ctx = __unw_seh_get_disp_ctx((unw_cursor_t *)context); +#if defined(__aarch64__) + LOCAL_DISPATCHER_CONTEXT_NONVOLREG

[libunwind] [libunwind] [SEH] Set NonVolatileRegisters before calling a personality function (PR #137951)

2025-05-04 Thread Jacek Caban via cfe-commits
https://github.com/cjacek edited https://github.com/llvm/llvm-project/pull/137951 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libunwind] [libunwind] [SEH] Implement parsing of ARM pdata/xdata (PR #137950)

2025-05-04 Thread Jacek Caban via cfe-commits
https://github.com/cjacek approved this pull request. LGTM https://github.com/llvm/llvm-project/pull/137950 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] raise extension warning for unknown namespaced attributes (PR #120925)

2025-05-04 Thread Oleksandr T. via cfe-commits
https://github.com/a-tarasyuk edited https://github.com/llvm/llvm-project/pull/120925 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
MightyFilipns wrote: > Please also update the release notes. It should be good now https://github.com/llvm/llvm-project/pull/137610 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Marco Elver via cfe-commits
@@ -1078,22 +1092,44 @@ void ModuleSanitizerCoverage::InjectCoverageAtBlock(Function &F, BasicBlock &BB, Store->setNoSanitizeMetadata(); } if (Options.StackDepth && IsEntryBB && !IsLeafFunc) { -// Check stack depth. If it's the deepest so far, record it. Modu

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Marco Elver via cfe-commits
https://github.com/melver requested changes to this pull request. This is also missing flag and IR tests. https://github.com/llvm/llvm-project/pull/138323 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/li

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Marco Elver via cfe-commits
https://github.com/melver edited https://github.com/llvm/llvm-project/pull/138323 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [sancov] Introduce optional callback for stack-depth tracking (PR #138323)

2025-05-04 Thread Marco Elver via cfe-commits
@@ -34,6 +34,7 @@ class SanitizerArgs { std::vector CoverageIgnorelistFiles; std::vector BinaryMetadataIgnorelistFiles; int CoverageFeatures = 0; + int StackDepthCallbackMin = 0; melver wrote: `CoverageStackDepthCallbackMin` https://github.com/llvm/llv

[clang] Attribute support `[[clang::musttail]]` in `ExprConstant.cpp` (work in progress) (PR #138477)

2025-05-04 Thread Hana Dusíková via cfe-commits
hanickadot wrote: ```c++ constexpr int deep_test(int remaining) { if (remaining == 0) { return 42; } MUST_TAIL return deep_test(remaining - 1); } constexpr int result = deep_test(200'000); ``` With debug build of clang: ```sh clang++ -std=c++26 -c deep.cp

[clang] Attribute support `[[clang::musttail]]` in `ExprConstant.cpp` (work in progress) (PR #138477)

2025-05-04 Thread Hana Dusíková via cfe-commits
https://github.com/hanickadot edited https://github.com/llvm/llvm-project/pull/138477 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Attribute support `[[clang::musttail]]` in `ExprConstant.cpp` (work in progress) (PR #138477)

2025-05-04 Thread Hana Dusíková via cfe-commits
https://github.com/hanickadot edited https://github.com/llvm/llvm-project/pull/138477 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Attribute support `[[clang::musttail]]` in `ExprConstant.cpp` (work in progress) (PR #138477)

2025-05-04 Thread Hana Dusíková via cfe-commits
https://github.com/hanickadot edited https://github.com/llvm/llvm-project/pull/138477 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Attribute support `[[clang::musttail]]` in `ExprConstant.cpp` (work in progress) (PR #138477)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Hana Dusíková (hanickadot) Changes This change makes `[[clang::musttail]]` work. Function calls marked with this attribute won't use system stack, but will loop after nearest function call. The attribute is already very strick, and checks

[clang] Attribute support `[[clang::musttail]]` in `ExprConstant.cpp` (work in progress) (PR #138477)

2025-05-04 Thread Hana Dusíková via cfe-commits
https://github.com/hanickadot created https://github.com/llvm/llvm-project/pull/138477 This change makes `[[clang::musttail]]` work. Function calls marked with this attribute won't use system stack, but will loop after nearest function call. The attribute is already very strick, and checks all

[clang] [clang] Remove unused local variables (NFC) (PR #138468)

2025-05-04 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `openmp-clang-x86_64-linux-debian` running on `gribozavr4` while building `clang` at step 6 "test-openmp". Full details are available at: https://lab.llvm.org/buildbot/#/builders/6/builds/8904 Here is the relevant piece of t

[clang] [clang] Remove unused local variables (NFC) (PR #138468)

2025-05-04 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata closed https://github.com/llvm/llvm-project/pull/138468 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 1812a43 - [clang] Remove unused local variables (NFC) (#138468)

2025-05-04 Thread via cfe-commits
Author: Kazu Hirata Date: 2025-05-04T14:14:23-07:00 New Revision: 1812a43a22a82adccf46d6589b8460a3a7a2678e URL: https://github.com/llvm/llvm-project/commit/1812a43a22a82adccf46d6589b8460a3a7a2678e DIFF: https://github.com/llvm/llvm-project/commit/1812a43a22a82adccf46d6589b8460a3a7a2678e.diff L

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
https://github.com/MightyFilipns updated https://github.com/llvm/llvm-project/pull/137610 >From dd0434829bebfa00493be35faa108b32efa62b03 Mon Sep 17 00:00:00 2001 From: MightyFilipns Date: Mon, 28 Apr 2025 11:54:20 +0200 Subject: [PATCH] clang-format: Add SpaceAfterOperatorKeyword option --- c

[clang] [llvm] Jay1 (PR #138469)

2025-05-04 Thread via cfe-commits
https://github.com/PatJay1703 closed https://github.com/llvm/llvm-project/pull/138469 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Jay1 (PR #138469)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (PatJay1703) Changes --- Patch is 45.90 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/138469.diff 10 Files Affected: - (modified) .clang-format (+245-1) - (added) clang-format-diff.p

[clang] [llvm] Jay1 (PR #138469)

2025-05-04 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] [llvm] Jay1 (PR #138469)

2025-05-04 Thread via cfe-commits
https://github.com/PatJay1703 created https://github.com/llvm/llvm-project/pull/138469 None >From 8f7b6fc86234162d9c88547c543552693c8b5c79 Mon Sep 17 00:00:00 2001 From: Jay Satish Kumar Patel Date: Sun, 4 May 2025 09:49:01 -0500 Subject: [PATCH 1/3] changed config file --- .clang-format | 2

[clang] [clang] Remove unused local variables (NFC) (PR #138468)

2025-05-04 Thread Nikita Popov via cfe-commits
https://github.com/nikic approved this pull request. https://github.com/llvm/llvm-project/pull/138468 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Remove unused local variables (NFC) (PR #138468)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-static-analyzer-1 Author: Kazu Hirata (kazutakahirata) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/138468.diff 7 Files Affected: - (modified) clang/include/clang/StaticAnalyzer/Core/PathSensitive/SMTConstraintManager.h (

[clang] [clang] Remove unused local variables (NFC) (PR #138468)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/138468.diff 7 Files Affected: - (modified) clang/include/clang/StaticAnalyzer/Core/PathSensitive/SMTConstraintManager.h (-2) - (modified)

[clang] [clang] Remove unused local variables (NFC) (PR #138468)

2025-05-04 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/138468 None >From 89189fd5a12716b35019271a7473887867b2c3e1 Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sun, 4 May 2025 13:03:48 -0700 Subject: [PATCH] [clang] Remove unused local variables (NFC) --- .../

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
@@ -17630,6 +17630,10 @@ TEST_F(FormatTest, ConfigurableSpaceBeforeParens) { verifyFormat("int x = int (y);", SomeSpace2); verifyFormat("auto lambda = []() { return 0; };", SomeSpace2); + FormatStyle SpaceAfterOperatorKeyword = getLLVMStyle(); + SpaceAfterOperatorKeyword

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
@@ -17630,6 +17630,10 @@ TEST_F(FormatTest, ConfigurableSpaceBeforeParens) { verifyFormat("int x = int (y);", SomeSpace2); verifyFormat("auto lambda = []() { return 0; };", SomeSpace2); + FormatStyle SpaceAfterOperatorKeyword = getLLVMStyle(); + SpaceAfterOperatorKeyword

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
@@ -17630,6 +17630,10 @@ TEST_F(FormatTest, ConfigurableSpaceBeforeParens) { verifyFormat("int x = int (y);", SomeSpace2); verifyFormat("auto lambda = []() { return 0; };", SomeSpace2); + FormatStyle SpaceAfterOperatorKeyword = getLLVMStyle(); MightyFilip

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
@@ -4463,6 +4463,14 @@ struct FormatStyle { /// \version 9 bool SpaceAfterLogicalNot; + /// If \c true, a space will be inserted after the ``operator`` keyword. + /// \code + ///true:false: + ///bool operator == (int a) vs.

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
@@ -4463,6 +4463,14 @@ struct FormatStyle { /// \version 9 bool SpaceAfterLogicalNot; + /// If \c true, a space will be inserted after the ``operator`` keyword. MightyFilipns wrote: Fixed https://github.com/llvm/llvm-project/pull/137610

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
@@ -4463,6 +4463,14 @@ struct FormatStyle { /// \version 9 bool SpaceAfterLogicalNot; + /// If \c true, a space will be inserted after the ``operator`` keyword. + /// \code + ///true:false: + ///bool operator == (int a) vs.

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
https://github.com/MightyFilipns updated https://github.com/llvm/llvm-project/pull/137610 Rate limit · GitHub body { background-color: #f6f8fa; color: #24292e; font-family: -apple-system,BlinkMacSystemFont,Segoe UI,Helvetica,Arial,

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
https://github.com/MightyFilipns updated https://github.com/llvm/llvm-project/pull/137610 >From d18481a5535c64267bd00c84252a81d9dc6baecd Mon Sep 17 00:00:00 2001 From: MightyFilipns Date: Mon, 28 Apr 2025 11:54:20 +0200 Subject: [PATCH] clang-format: Add SpaceAfterOperatorKeyword option --- c

[clang] [clang][Driver][test] Fix Driver/solaris-ld.c on non-Unix hosts (PR #138466)

2025-05-04 Thread Rainer Orth via cfe-commits
https://github.com/rorth closed https://github.com/llvm/llvm-project/pull/138466 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 142f99a - [clang][Driver][test] Fix Driver/solaris-ld.c (#138466)

2025-05-04 Thread via cfe-commits
Author: Rainer Orth Date: 2025-05-04T21:44:56+02:00 New Revision: 142f99ad645f9e77dd5adb470e41c55ce6d37b2b URL: https://github.com/llvm/llvm-project/commit/142f99ad645f9e77dd5adb470e41c55ce6d37b2b DIFF: https://github.com/llvm/llvm-project/commit/142f99ad645f9e77dd5adb470e41c55ce6d37b2b.diff L

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
https://github.com/MightyFilipns updated https://github.com/llvm/llvm-project/pull/137610 >From 601985f75cf961f585a3d232868eda3a400878fc Mon Sep 17 00:00:00 2001 From: MightyFilipns Date: Mon, 28 Apr 2025 11:54:20 +0200 Subject: [PATCH] clang-format: Add SpaceAfterOperatorKeyword option --- c

[clang] [clang][Driver][test] Fix Driver/solaris-ld.c on non-Unix hosts (PR #138466)

2025-05-04 Thread Rainer Orth via cfe-commits
rorth wrote: > It's not due to non-Unix hosts. It's likely due to these builds setting > CLANG_DEFAULT_RTLIB and CLANG_DEFAULT_UNWINDLIB. That occured to me shortly after I created the PR. I'll fix the description accordingly. https://github.com/llvm/llvm-project/pull/138466

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
https://github.com/MightyFilipns updated https://github.com/llvm/llvm-project/pull/137610 >From aabc71f63914ec58e312fc89f6102919333f0b7f Mon Sep 17 00:00:00 2001 From: MightyFilipns Date: Mon, 28 Apr 2025 11:54:20 +0200 Subject: [PATCH] clang-format: Add SpaceAfterOperatorKeyword option # Conf

[clang] [clang][Driver][test] Fix Driver/solaris-ld.c on non-Unix hosts (PR #138466)

2025-05-04 Thread Fangrui Song via cfe-commits
https://github.com/MaskRay approved this pull request. https://github.com/llvm/llvm-project/pull/138466 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][Driver][test] Fix Driver/solaris-ld.c on non-Unix hosts (PR #138466)

2025-05-04 Thread Fangrui Song via cfe-commits
MaskRay wrote: It's not due to non-Unix hosts. It's likely due to these builds setting CLANG_DEFAULT_RTLIB and CLANG_DEFAULT_UNWINDLIB. https://github.com/llvm/llvm-project/pull/138466 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://li

[clang] [clang-format] Add SpaceAfterOperatorKeyword option (PR #137610)

2025-05-04 Thread Filip Milosevic via cfe-commits
https://github.com/MightyFilipns updated https://github.com/llvm/llvm-project/pull/137610 >From 2bbc9f699de5e73d899c5d58607ccfe36aa7a7d7 Mon Sep 17 00:00:00 2001 From: MightyFilipns Date: Mon, 28 Apr 2025 11:54:20 +0200 Subject: [PATCH] clang-format: Add SpaceAfterOperatorKeyword option --- c

[clang] [clang-format] Don't annotate enum colon as InheritanceColon (PR #138440)

2025-05-04 Thread Björn Schäpers via cfe-commits
@@ -1423,7 +1423,7 @@ class AnnotatingParser { Tok->setType(TT_BitFieldColon); } else if (Contexts.size() == 1 && !Line.First->isOneOf(tok::kw_enum, tok::kw_case, - tok::kw_default)) { +

[clang] [Driver] Use AddRunTimeLibs on Solaris (PR #137596)

2025-05-04 Thread Rainer Orth via cfe-commits
rorth wrote: I've just created PR #138466 which hopefully fixes those failures. Can anyone affected please give this a try? https://github.com/llvm/llvm-project/pull/137596 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.or

[clang] [clang][Driver][test] Fix Driver/solaris-ld.c on non-Unix hosts (PR #138466)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Rainer Orth (rorth) Changes `Clang :: Driver/solaris-ld.c` currently `FAIL`s on two non-Unix buildbots: [fuchsia-x86_64-linux](https://lab.llvm.org/buildbot/#/builders/11/builds/14369) and [llvm-clang-win-x-aarch64](https://lab.llvm.org/b

[clang] [clang][Driver][test] Fix Driver/solaris-ld.c on non-Unix hosts (PR #138466)

2025-05-04 Thread Rainer Orth via cfe-commits
https://github.com/rorth created https://github.com/llvm/llvm-project/pull/138466 `Clang :: Driver/solaris-ld.c` currently `FAIL`s on two non-Unix buildbots: [fuchsia-x86_64-linux](https://lab.llvm.org/buildbot/#/builders/11/builds/14369) and [llvm-clang-win-x-aarch64](https://lab.llvm.org/bui

[clang] 4c587f5 - [clang] Use *(Set|Map)::contains (NFC) (#138464)

2025-05-04 Thread via cfe-commits
Author: Kazu Hirata Date: 2025-05-04T11:56:31-07:00 New Revision: 4c587f549cbbea0ac1db2b983b8c5fb4fff82d72 URL: https://github.com/llvm/llvm-project/commit/4c587f549cbbea0ac1db2b983b8c5fb4fff82d72 DIFF: https://github.com/llvm/llvm-project/commit/4c587f549cbbea0ac1db2b983b8c5fb4fff82d72.diff L

[clang] [clang] Use *(Set|Map)::contains (NFC) (PR #138464)

2025-05-04 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata closed https://github.com/llvm/llvm-project/pull/138464 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Use *(Set|Map)::contains (NFC) (PR #138464)

2025-05-04 Thread Balazs Benics via cfe-commits
https://github.com/steakhal approved this pull request. https://github.com/llvm/llvm-project/pull/138464 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Add support for Debian 14 Forky and Debian 15 Duke (PR #138460)

2025-05-04 Thread LLVM Continuous Integration via cfe-commits
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/17214 Her

[clang] [clang] Use *(Set|Map)::contains (NFC) (PR #138464)

2025-05-04 Thread Matt Arsenault via cfe-commits
https://github.com/arsenm approved this pull request. https://github.com/llvm/llvm-project/pull/138464 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Use *(Set|Map)::contains (NFC) (PR #138464)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-modules Author: Kazu Hirata (kazutakahirata) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/138464.diff 4 Files Affected: - (modified) clang/lib/AST/DeclBase.cpp (+1-2) - (modified) clang/lib/Serialization/ASTReader.cpp (+2-

[clang] [clang] Use *(Set|Map)::contains (NFC) (PR #138464)

2025-05-04 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/138464.diff 4 Files Affected: - (modified) clang/lib/AST/DeclBase.cpp (+1-2) - (modified) clang/lib/Serialization/ASTReader.cpp (+2-3) - (m

[clang] [clang] Use *(Set|Map)::contains (NFC) (PR #138464)

2025-05-04 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/138464 None >From 988dae3f61b85a357f271336695619060103aeca Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Sun, 4 May 2025 10:41:49 -0700 Subject: [PATCH] [clang] Use *(Set|Map)::contains (NFC) --- clang/lib

[clang] [clang] Add support for Debian 14 Forky and Debian 15 Duke (PR #138460)

2025-05-04 Thread Raul Tambre via cfe-commits
https://github.com/tambry closed https://github.com/llvm/llvm-project/pull/138460 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 58e6883 - [clang] Add support for Debian 14 Forky and Debian 15 Duke (#138460)

2025-05-04 Thread via cfe-commits
Author: Raul Tambre Date: 2025-05-04T20:55:49+03:00 New Revision: 58e6883c8b6e571d6bd774645ee2b6348cfed6ba URL: https://github.com/llvm/llvm-project/commit/58e6883c8b6e571d6bd774645ee2b6348cfed6ba DIFF: https://github.com/llvm/llvm-project/commit/58e6883c8b6e571d6bd774645ee2b6348cfed6ba.diff L

[clang] [clang] Remove unused local variables (NFC) (PR #138453)

2025-05-04 Thread Kazu Hirata via cfe-commits
https://github.com/kazutakahirata closed https://github.com/llvm/llvm-project/pull/138453 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] f002f30 - [clang] Remove unused local variables (NFC) (#138453)

2025-05-04 Thread via cfe-commits
Author: Kazu Hirata Date: 2025-05-04T10:51:40-07:00 New Revision: f002f300c5e1f9ce8adf00441b7e879fec36a071 URL: https://github.com/llvm/llvm-project/commit/f002f300c5e1f9ce8adf00441b7e879fec36a071 DIFF: https://github.com/llvm/llvm-project/commit/f002f300c5e1f9ce8adf00441b7e879fec36a071.diff L

[clang] [clang] Remove unused local variables (NFC) (PR #138453)

2025-05-04 Thread Kazu Hirata via cfe-commits
@@ -361,15 +361,8 @@ TEST(CallDescription, AliasNames) { std::cont v; v.data(); })code"; - constexpr StringRef UseStructNameInSpelling = R"code( -void foo() { - std::container v; - v.data(); -})code"; const std::string UseAliasInSpellingCod

[clang] [analyzer] Improve cache locality by using separate allocators (PR #138295)

2025-05-04 Thread Balazs Benics via cfe-commits
steakhal wrote: I had a look at the spreadsheet and the numbers for `linux-topro.c` looks really impressive. However, in general, we need further investigation about why are we not getting faster e.g. in the `clang-MSP430ISelDAGToDAG.cpp` case. The direction seems right though. (FYI it usuall

[clang] [clang][bytecode] Use bytecode interpreter in EvaluateCharRangeAsString (PR #138461)

2025-05-04 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr updated https://github.com/llvm/llvm-project/pull/138461 >From 6cc6ff2f4c62e6d0a7b95338c461f8998a4a6bbc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Sun, 4 May 2025 07:25:20 +0200 Subject: [PATCH] [clang][bytecode] Use bytecode interpreter in E

[clang] [analyzer] Improve cache locality by using separate allocators (PR #138295)

2025-05-04 Thread Balazs Benics via cfe-commits
https://github.com/steakhal edited https://github.com/llvm/llvm-project/pull/138295 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

  1   2   >