[clang] [llvm] [Hexagon] Add missing builtins for V79 (PR #122916)

https://github.com/androm3da approved this pull request. https://github.com/llvm/llvm-project/pull/122916 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Fix modernize-use-integer-sign-comparison comparison (PR #121506)

https://github.com/qt-tatiana updated https://github.com/llvm/llvm-project/pull/121506 >From 0591e8b7be49299e2b73634a6ad4f2330557b37a Mon Sep 17 00:00:00 2001 From: Tatiana Borisova Date: Thu, 2 Jan 2025 18:08:26 +0100 Subject: [PATCH 1/3] [clang-tidy] Fix modernize-use-integer-sign-comparison

[clang] [llvm] [Clang] restrict use of attribute names reserved by the C++ standard (PR #106036)

AaronBallman wrote: > IN SUMMARY: I think this proposal minus the std check is acceptable. +1 > I think the proposed-not-implemented warning of "you messed with an attribute > name, and it is doing weird stuff" is also a very valuable, though different, > check. I don't see them as different

[clang] [CUDA][HIP] Support CUID in new driver (PR #122859)

yxsamliu wrote: > > > I thought the new driver just got some entropy from the current source > > > file's `inode` entry? I'm not opposed, since I guess it would make sense > > > to give the user a way to override it. > > > > > > Sorry I did not quite get your question. Do you suggest that we

[clang] [CUDA][HIP] Support CUID in new driver (PR #122859)

jhuber6 wrote: > > > > I thought the new driver just got some entropy from the current source > > > > file's `inode` entry? I'm not opposed, since I guess it would make > > > > sense to give the user a way to override it. > > > > > > > > > Sorry I did not quite get your question. Do you sugge

[clang] [flang] [flang] Add -f[no-]unroll-loops flag (PR #122906)

@@ -0,0 +1,37 @@ +! RUN: %flang_fc1 -emit-llvm -O1 -funroll-loops -mllvm -force-vector-width=2 -o- %s | FileCheck %s --check-prefixes=CHECK,UNROLL +! RUN: %flang_fc1 -emit-llvm -O2 -mllvm -force-vector-width=2 -o- %s | FileCheck %s --check-prefixes=CHECK,UNROLL +! RUN: %flang_fc

[clang-tools-extra] [clang-tidy][modernize] Replace memmove/memcpy with std::copy (PR #122940)

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] [flang] [flang] Add -f[no-]unroll-loops flag (PR #122906)

https://github.com/DavidTruby updated https://github.com/llvm/llvm-project/pull/122906 >From c9b2e5855fdbbaafb5512e1e2539983201202b25 Mon Sep 17 00:00:00 2001 From: David Truby Date: Wed, 8 Jan 2025 11:19:38 + Subject: [PATCH 1/5] [flang] Add -f[no-]unroll-loops flag This patch adds suppor

[clang] [clang-tools-extra] [lldb] [Clang] [NFC] Rename `isAnyPointerType()` and `getPointeeOrArrayElementType()`. (PR #122938)

cor3ntin wrote: Thanks for working on this. A few comment: `getPointerLikeOrArrayElementType` is not used, and I'm rather concerned by what it does. Removing pointiness in some cases seem dangerous, and not super useful... Renaming `getPointeeOrArrayElementType` to `getPointerOrArrayElementTy

[clang] [flang] [flang] Add -f[no-]unroll-loops flag (PR #122906)

https://github.com/kiranchandramohan edited https://github.com/llvm/llvm-project/pull/122906 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][bytecode] Handle UETT_PtrAuthTypeDiscriminator (PR #122941)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/122941.diff 2 Files Affected: - (modified) clang/lib/AST/ByteCode/Compiler.cpp (+10) - (modified) clang/test/SemaCXX/ptrauth-type-discriminator.cp

[clang] a0bd40e - Revert "[clang][bytecode] Mark global decls with diagnostics invalid (#122895)"

Author: Timm Bäder Date: 2025-01-14T18:37:36+01:00 New Revision: a0bd40e5a3df94229ec06243f2958289071ca75c URL: https://github.com/llvm/llvm-project/commit/a0bd40e5a3df94229ec06243f2958289071ca75c DIFF: https://github.com/llvm/llvm-project/commit/a0bd40e5a3df94229ec06243f2958289071ca75c.diff LO

[clang] [clang][bytecode] Handle UETT_PtrAuthTypeDiscriminator (PR #122941)

https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/122941 None >From 4bef28263ea3947b3bc80835fde340c776d0aad3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Tue, 14 Jan 2025 18:36:30 +0100 Subject: [PATCH] [clang][bytecode] Handle UETT_PtrAuthType

[clang] [CUDA][HIP] Support CUID in new driver (PR #122859)

jhuber6 wrote: > That would break rccl. We should use CUID's that work in general, including > apps that compile the same file with different options. Shouldn't be broken if the driver handles the CUID, I just mean that it would work in `cc1` if not passed. https://github.com/llvm/llvm-proj

[clang] [clang-tools-extra] [lldb] [Clang] [NFC] Rename `isAnyPointerType()` and `getPointeeOrArrayElementType()`. (PR #122938)

Sirraide wrote: > A few comment: `getPointerLikeOrArrayElementType` is not used, and I'm rather > concerned by what it does. > Removing pointiness in only some cases seem dangerous, and not super useful... That’s fair, I don’t think it’s crucial to add that or anything. > Renaming `getPointeeO

[clang] [clang][Sema] Move computing enum width and type to a separate function (PR #120965)

https://github.com/kuilpd updated https://github.com/llvm/llvm-project/pull/120965 >From ab8dcaef120233d0145508aaa4bf45eec22cadf1 Mon Sep 17 00:00:00 2001 From: Ilia Kuklin Date: Mon, 23 Dec 2024 17:49:32 +0500 Subject: [PATCH 1/4] [clang][Sema] Move calculating enum width and type to a separa

[clang] [clang-tools-extra] [lldb] [Clang] [NFC] Rename `isAnyPointerType()` and `getPointeeOrArrayElementType()`. (PR #122938)

Sirraide wrote: > That’s fair, I don’t think it’s crucial to add that or anything. If anything the pr in question can either add that or put that somewhere else; I’ll remove that function from this pr. https://github.com/llvm/llvm-project/pull/122938 ___

[clang] [CUDA][HIP] Support CUID in new driver (PR #122859)

yxsamliu wrote: > > Yeah, that's fine, but we should just default to the inode method if the > argument isn't present I think. That way it works without needing this extra > argument passed to both the compile phases. That would break rccl. We should use CUID's that work in general, including

[clang] [clang-tools-extra] [lldb] [Clang] [NFC] Rename `isAnyPointerType()` and `getPointeeOrArrayElementType()`. (PR #122938)

Sirraide wrote: (just removed the last two commits that only added that function) https://github.com/llvm/llvm-project/pull/122938 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][Sema] Move computing enum width and type to a separate function (PR #120965)

kuilpd wrote: Tried moving the code to `EnumDecl`, but it didn't work as well as I thought, so moved it to `ASTContext` after all. Don't really know where exactly to place it inside `ASTContext.h`, though. https://github.com/llvm/llvm-project/pull/120965 ___

[clang] [clang-tools-extra] [lldb] [Clang] [NFC] Rename `isAnyPointerType()` and `getPointeeOrArrayElementType()`. (PR #122938)

cor3ntin wrote: > Ah, I think you’re confusing two functions there: > `getPointeeOrArrayElementType()` is being renamed to > `getPointerOrObjCPointerOrArrayElementType()`, not > `getPointerOrArrayElementType()`. Not confused. renaming `getPointeeFoo` to `getPointerBar` does not seem correct,

[clang] [clang-tools-extra] [lldb] [Clang] [NFC] Rename `isAnyPointerType()` and `getPointeeOrArrayElementType()`. (PR #122938)

Sirraide wrote: > Not confused. renaming `getPointeeFoo` to `getPointerBar` does not seem > correct, especially as `getPointeeOrArrayElementType` does indeed return a > pointee. Ah, I see, in that case the only other name I could think of would be something like `getPointerOrObjCPointerPointe

[clang] [llvm] [SPIRV] add pre legalization instruction combine (PR #122839)

https://github.com/farzonl converted_to_draft https://github.com/llvm/llvm-project/pull/122839 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Make armv6 iOS by default instead of macOS when targeting Darwin (PR #117243)

https://github.com/Un1q32 updated https://github.com/llvm/llvm-project/pull/117243 >From 4a746cbe467f4ad22436dc206a8966ecaa77892f Mon Sep 17 00:00:00 2001 From: Un1q32 Date: Thu, 21 Nov 2024 17:31:07 -0500 Subject: [PATCH] make armv6-darwin iOS by default --- clang/lib/Driver/ToolChains/Darwi

[clang-tools-extra] [clang-tidy] Add a release note about unchecked-optional-access smart pointer caching (PR #122290)

@@ -239,6 +239,10 @@ Changes in existing checks ` to support ``bsl::optional`` and ``bdlb::NullableValue`` from _. + Fixed false positives from smart pointer accessors repeated in checking + ``has_value`` and accessing ``value``, by cac

[clang-tools-extra] [clang-tidy] Add a release note about unchecked-optional-access smart pointer caching (PR #122290)

@@ -232,6 +232,10 @@ Changes in existing checks ` to support `bsl::optional` and `bdlb::NullableValue` from _. + Fixed false positives from smart pointer accessors repeated in checking + ``has_value`` and accessing ``value``, by caching

[clang-tools-extra] [clang-tidy] Refactor: removed typos in 'AllowedTypes' option in various checks (PR #122957)

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] Refactor: removed typos in 'AllowedTypes' option in various checks (PR #122957)

llvmbot wrote: @llvm/pr-subscribers-clang-tidy Author: Baranov Victor (vbvictor) Changes Added right parenthesis to match left one. --- Full diff: https://github.com/llvm/llvm-project/pull/122957.diff 3 Files Affected: - (modified) clang-tools-extra/docs/clang-tidy/checks/performance/

[clang-tools-extra] [clang-tidy] Refactor: removed typos in 'AllowedTypes' option in various checks (PR #122957)

https://github.com/vbvictor created https://github.com/llvm/llvm-project/pull/122957 Added right parenthesis to match left one. >From 9c49e1e558c43d13872afe0a10345e3510afebdf Mon Sep 17 00:00:00 2001 From: Victor Baranov Date: Tue, 14 Jan 2025 22:30:03 +0300 Subject: [PATCH] [clang-tidy] Fix:

[clang] Add 'enum_select' diagnostic selection to clang. (PR #122505)

https://github.com/AaronBallman commented: Thank you for working on this! We should probably also update https://clang.llvm.org/docs/InternalsManual.html#formatting-a-diagnostic-argument so that folks know how to use this properly and we advertise it more broadly. https://github.com/llvm/llvm-

[clang] Add 'enum_select' diagnostic selection to clang. (PR #122505)

@@ -886,6 +909,11 @@ struct DiagTextDocPrinter : DiagTextVisitor { makeRowSeparator(RST[I]); } + void VisitEnumSelect(EnumSelectPiece *P) { +// For now, document this as if it were a 'select'. AaronBallman wrote: Should we file an issue to addre

[clang] Add 'enum_select' diagnostic selection to clang. (PR #122505)

@@ -0,0 +1,16 @@ +// RUN: clang-tblgen --gen-clang-diags-enums -I%S %s 2>&1 | FileCheck %s +include "DiagnosticBase.inc" + +def DupeNames1 : Error<"%enum_select{}0">; +def DupeNames2 : Error<"%enum_select{}0">; +// CHECK: error: Duplicate enumeration name 'DupeName' +// CHECK-NEXT

[clang] Add 'enum_select' diagnostic selection to clang. (PR #122505)

@@ -1042,6 +1119,9 @@ Piece *DiagnosticTextBuilder::DiagText::parseDiagText(StringRef &Text, Modifier); }; +if (ModType != MT_EnumSelect && Text[0] == '<') + Builder.PrintFatalError("modifier '<' syntax not valid with " + Modifier

[clang] Add 'enum_select' diagnostic selection to clang. (PR #122505)

@@ -0,0 +1,16 @@ +// RUN: clang-tblgen --gen-clang-diags-enums -I%S %s 2>&1 | FileCheck %s +include "DiagnosticBase.inc" + +def DupeNames1 : Error<"%enum_select{}0">; AaronBallman wrote: Other test cases: ``` // Missing closing > after the name of the enumeration

[clang-tools-extra] [clang-tidy] Refactor: removed typos in 'AllowedTypes' option in various checks (PR #122957)

llvmbot wrote: @llvm/pr-subscribers-clang-tools-extra Author: Baranov Victor (vbvictor) Changes Added right parenthesis to match left one. --- Full diff: https://github.com/llvm/llvm-project/pull/122957.diff 3 Files Affected: - (modified) clang-tools-extra/docs/clang-tidy/checks/perfo

[clang] Add 'enum_select' diagnostic selection to clang. (PR #122505)

https://github.com/AaronBallman edited https://github.com/llvm/llvm-project/pull/122505 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

https://github.com/vbvictor edited https://github.com/llvm/llvm-project/pull/122951 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Add 'enum_select' diagnostic selection to clang. (PR #122505)

@@ -0,0 +1,16 @@ +// RUN: clang-tblgen --gen-clang-diags-enums -I%S %s 2>&1 | FileCheck %s +include "DiagnosticBase.inc" + +def DupeNames1 : Error<"%enum_select{}0">; +def DupeNames2 : Error<"%enum_select{}0">; +// CHECK: error: Duplicate enumeration name 'DupeName' +// CHECK-NEXT

[clang-tools-extra] [clang-tidy] Refactor: removed typos in 'AllowedTypes' option in various checks (PR #122957)

vbvictor wrote: @HerrCai0907, Could you look at this PR, please. I have seen you doing some refactor/cleanups lately. https://github.com/llvm/llvm-project/pull/122957 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bi

[clang] [clang][AST] Assert that DependentNameType::Name is not null (PR #122418)

https://github.com/mizvekov approved this pull request. LGTM, Indeed a DependentNameType with no 'Name' doesn't make sense. https://github.com/llvm/llvm-project/pull/122418 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/

[clang] [clang][AST] Assert that DependentNameType::Name is not null (PR #122418)

@@ -7035,17 +7035,16 @@ class DependentNameType : public TypeWithKeyword, public llvm::FoldingSetNode { : TypeWithKeyword(Keyword, DependentName, CanonType, TypeDependence::DependentInstantiation | toTypeDependence(NNS-

[clang] [clang][AST] Assert that DependentNameType::Name is not null (PR #122418)

https://github.com/mizvekov edited https://github.com/llvm/llvm-project/pull/122418 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [HLSL] cbuffer: Create host layout struct and add resource handle to AST (PR #122820)

https://github.com/hekota updated https://github.com/llvm/llvm-project/pull/122820 >From 71ddb5a2b4cc8a9609410b436e896484401f5e90 Mon Sep 17 00:00:00 2001 From: Helena Kotas Date: Mon, 13 Jan 2025 15:03:12 -0800 Subject: [PATCH 1/2] [HLSL] cbuffer: Create host layout struct and add resource ha

[clang] [clang][Sema] Move computing enum width and type to a separate function (PR #120965)

https://github.com/kuilpd updated https://github.com/llvm/llvm-project/pull/120965 >From ab8dcaef120233d0145508aaa4bf45eec22cadf1 Mon Sep 17 00:00:00 2001 From: Ilia Kuklin Date: Mon, 23 Dec 2024 17:49:32 +0500 Subject: [PATCH 1/5] [clang][Sema] Move calculating enum width and type to a separa

[clang] Add Clang attribute to ensure that fields are initialized explicitly (PR #102040)

higher-performance wrote: Ah haha I see, thanks! Looking forward to getting it sometime, that would definitely be helpful :) https://github.com/llvm/llvm-project/pull/102040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.or

[clang] Support [[guarded_by(mutex)]] attribute inside C struct (PR #94216)

bvanassche wrote: Thread-safety attributes of cleanup functions are ignored. Does anyone know how to fix this? See also https://github.com/llvm/llvm-project/issues/122758. https://github.com/llvm/llvm-project/pull/94216 ___ cfe-commits mailing list cf

[clang] Add Clang attribute to ensure that fields are initialized explicitly (PR #102040)

AaronBallman wrote: > Fantastic, thank you! Could we merge it? heh, thanks for the poke, I didn't remember you didn't have commit access. I've merged now. https://github.com/llvm/llvm-project/pull/102040 ___ cfe-commits mailing list cfe-commits@lists

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

@@ -179,6 +179,8 @@ def err_opencl_unknown_type_specifier : Error< def warn_unknown_attribute_ignored : Warning< "unknown attribute %0 ignored">, InGroup; +def ext_unknown_attribute_ignored : Extension< + "unknown attribute %0 ignored">, InGroup; AaronBallm

[clang] Add Clang attribute to ensure that fields are initialized explicitly (PR #102040)

higher-performance wrote: @AaronBallman I... just noticed that the name `clang::requires_explicit_initialization` is subtly different from `clang::require_constant_initialization`. (`requires` vs. `require`.) Is it worth making it consistent before people start using it? I can submit a PR qui

[clang] [llvm] [clang] Implement pragma clang section on COFF targets (PR #112714)

@@ -1684,6 +1676,7 @@ MCSection *TargetLoweringObjectFileCOFF::getExplicitSectionGlobal( Name == getInstrProfSectionName(IPSK_covname, Triple::COFF, /*AddSegmentInfo=*/false)) Kind = SectionKind::getMetadata(); +

[compiler-rt] [libunwind] [libunwind][cmake] Compile _Unwind* routines with -fexceptions (PR #121819)

ilovepi wrote: Are there lingering concerns here or are we OK landing this? https://github.com/llvm/llvm-project/pull/121819 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Refactor: removed typos in 'AllowedTypes' option in various checks (PR #122957)

https://github.com/vbvictor edited https://github.com/llvm/llvm-project/pull/122957 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[compiler-rt] [libunwind] [libunwind][cmake] Compile _Unwind* routines with -fexceptions (PR #121819)

https://github.com/ilovepi edited https://github.com/llvm/llvm-project/pull/121819 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Comments] Allow HTML tags across multiple lines (PR #120843)

https://github.com/AaronBallman commented: Please be sure to also add a release note to `clang/docs/ReleaseNotes.rst` so users know about the fix. In principle, the changes LGTM but I do wonder if this is a part of the compiler that could benefit from some fuzzing to help shake out parsing iss

[clang] [llvm] [HLSL][SPIRV][DXIL] Implement `WaveActiveSum` intrinsic (PR #118580)

https://github.com/adam-yang updated https://github.com/llvm/llvm-project/pull/118580 >From 9498523511a53c43b6282500fed2659e12ddd752 Mon Sep 17 00:00:00 2001 From: Finn Plummer Date: Fri, 18 Oct 2024 10:49:18 -0700 Subject: [PATCH 1/7] [HLSL][SPIRV][DXIL] Implement `WaveActiveSum` intrinsic -

[clang] 1594413 - Add Clang attribute to ensure that fields are initialized explicitly (#102040)

Author: higher-performance Date: 2025-01-14T13:31:12-05:00 New Revision: 1594413d5edf6a47d4100cb6a2bc613cfbb92beb URL: https://github.com/llvm/llvm-project/commit/1594413d5edf6a47d4100cb6a2bc613cfbb92beb DIFF: https://github.com/llvm/llvm-project/commit/1594413d5edf6a47d4100cb6a2bc613cfbb92beb.

[clang] Add Clang attribute to ensure that fields are initialized explicitly (PR #102040)

https://github.com/AaronBallman closed https://github.com/llvm/llvm-project/pull/102040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 71f238a - [Hexagon] Add missing builtins for V79 (#122916)

Author: Ikhlas Ajbar Date: 2025-01-14T12:21:40-06:00 New Revision: 71f238a221ff307a1c079678b85ef3abe43c71ab URL: https://github.com/llvm/llvm-project/commit/71f238a221ff307a1c079678b85ef3abe43c71ab DIFF: https://github.com/llvm/llvm-project/commit/71f238a221ff307a1c079678b85ef3abe43c71ab.diff

[clang] [llvm] [Hexagon] Add missing builtins for V79 (PR #122916)

https://github.com/iajbar closed https://github.com/llvm/llvm-project/pull/122916 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Add Clang attribute to ensure that fields are initialized explicitly (PR #102040)

https://github.com/AaronBallman approved this pull request. LGTM, thank you for your patience on the review! https://github.com/llvm/llvm-project/pull/102040 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman

[clang] [clang][Sema] Move computing enum width and type to a separate function (PR #120965)

Sirraide wrote: > Don't really know where exactly to place it inside `ASTContext.h`, though. Honestly, just put it wherever I’d say. Unlike `Sema.h`, `ASTContext.h` is much less well-organised last time I checked. https://github.com/llvm/llvm-project/pull/120965 ___

[clang-tools-extra] [clang-tidy] Refactor: removed typos in 'AllowedTypes' option in various checks (PR #122957)

https://github.com/HerrCai0907 approved this pull request. https://github.com/llvm/llvm-project/pull/122957 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [libclang] Add API to query more information about base classes. (PR #120300)

efriedma-quic wrote: The 32-bit Arm failures are fixed by 7aec7caca30f800811b76ba94291645494788a4f . The other failures appear to be unrelated. https://github.com/llvm/llvm-project/pull/120300 ___ cfe-commits mailing list cfe-commits@lists.llvm.org h

[clang] [clang][DebugInfo] Emit DW_AT_object_pointer on function definitions with explicit `this` (PR #122897)

llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `llvm-clang-win-x-armv7l` running on `as-builder-1` while building `clang` at step 15 "test-check-cxx-armv7-unknown-linux-gnueabihf". Full details are available at: https://lab.llvm.org/buildbot/#/builders/38/builds/1830 Her

[clang-tools-extra] bc74625 - [clang-tidy] Add an option to exclude files not present in the compile database (#120348)

Author: zotnhucucbot Date: 2025-01-15T07:59:03+08:00 New Revision: bc74625f50e216edd16f436c4fc81ff585b6c4c7 URL: https://github.com/llvm/llvm-project/commit/bc74625f50e216edd16f436c4fc81ff585b6c4c7 DIFF: https://github.com/llvm/llvm-project/commit/bc74625f50e216edd16f436c4fc81ff585b6c4c7.diff

[clang-tools-extra] [clang-tidy] Add an option to exclude files not present in the compile database (PR #120348)

https://github.com/HerrCai0907 closed https://github.com/llvm/llvm-project/pull/120348 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clangd] Support .clangd command line modifications for C++ modules (PR #122606)

https://github.com/ChuanqiXu9 approved this pull request. Thanks. This looks good to me. And please give @kadircet some time to give it a look. I'll try to land this in the next week if no more comments came in. https://github.com/llvm/llvm-project/pull/122606 __

[clang-tools-extra] [clangd] Support .clangd command line modifications for C++ modules (PR #122606)

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 657fb4433e027722e8c9a5002d0c194ecd3f2956 5c89b9dbf97b57d232817a4716433239e84365a1 --e

[clang] [Modules] Delay deserialization of preferred_name attribute at r… (PR #122726)

https://github.com/ChuanqiXu9 commented: Thanks. (1) Hardcoding is generally not good. And even if we have to, e.g., (we don't have an automatic mechanism to check if we need to defer an attribute), let's avoid hardcoding in both reader side and writer side. We can make this by adding a new b

[clang] [Modules] Delay deserialization of preferred_name attribute at r… (PR #122726)

@@ -1205,6 +1205,23 @@ class ASTReader /// been completed. std::deque PendingDeclContextInfos; + /// Deserialization of some attributes must be deferred since they refer + /// to themselves in their type (e.g., preferred_name attribute refers to the + /// typedef that r

[clang] [Modules] Delay deserialization of preferred_name attribute at r… (PR #122726)

https://github.com/ChuanqiXu9 edited https://github.com/llvm/llvm-project/pull/122726 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Modules] Delay deserialization of preferred_name attribute at r… (PR #122726)

@@ -1205,6 +1205,23 @@ class ASTReader /// been completed. std::deque PendingDeclContextInfos; + /// Deserialization of some attributes must be deferred since they refer + /// to themselves in their type (e.g., preferred_name attribute refers to the + /// typedef that r

[clang] d1d2564 - [clang][bytecode] Handle UETT_PtrAuthTypeDiscriminator (#122941)

Author: Timm Baeder Date: 2025-01-15T08:41:20+01:00 New Revision: d1d25641f4cb87ab2c07a4136ba1cec4fb6cf578 URL: https://github.com/llvm/llvm-project/commit/d1d25641f4cb87ab2c07a4136ba1cec4fb6cf578 DIFF: https://github.com/llvm/llvm-project/commit/d1d25641f4cb87ab2c07a4136ba1cec4fb6cf578.diff L

[clang] [clang][bytecode] Handle UETT_PtrAuthTypeDiscriminator (PR #122941)

https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/122941 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Correct the order of substituted arguments in CTAD alias guides (PR #123022)

https://github.com/zyn0217 created https://github.com/llvm/llvm-project/pull/123022 We missed a case of type constraints referencing deduced template parameters when constructing a deduction guide for the type alias. This patch fixes the issue by swapping the order of constructing 'template ar

[clang] [Hexagon] Add default clang symlinks to CLANG_LINKS_TO_CREATE (PR #123011)

https://github.com/androm3da created https://github.com/llvm/llvm-project/pull/123011 Since this cache value overrides the defaults, we end up with `clang` linked to `clang-20`, and some `${triple}-clang*` links, but we're missing `clang++`. This makes for a toolchain with inconsistent behavi

[clang] [Hexagon] Add default clang symlinks to CLANG_LINKS_TO_CREATE (PR #123011)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Brian Cain (androm3da) Changes Since this cache value overrides the defaults, we end up with `clang` linked to `clang-20`, and some `${triple}-clang*` links, but we're missing `clang++`. This makes for a toolchain with inconsistent behav

[clang] [clang-format] Support BraceWrapping.AfterNamespace with AllowShortNamespacesOnASingleLine (PR #123010)

llvmbot wrote: @llvm/pr-subscribers-clang-format Author: Galen Elias (galenelias) Changes AllowShortNamespacesOnASingleLine assumes that there is no newline before the namespace brace, however, there is no actual reason this shouldn't be compatible with BraceWrapping.AfterNamespace = tru

[clang-tools-extra] [clangd] Augment code completion results with documentation from the index. (PR #120099)

https://github.com/HighCommander4 approved this pull request. Thanks for the update! This looks good, I'll go ahead and merge it. https://github.com/llvm/llvm-project/pull/120099 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llv

[clang-tools-extra] 030d48b - [clangd] Augment code completion results with documentation from the index. (#120099)

Author: Bevin Hansson Date: 2025-01-15T01:14:20-05:00 New Revision: 030d48b7db9845f42bf3ef365193bfdbb23f5440 URL: https://github.com/llvm/llvm-project/commit/030d48b7db9845f42bf3ef365193bfdbb23f5440 DIFF: https://github.com/llvm/llvm-project/commit/030d48b7db9845f42bf3ef365193bfdbb23f5440.diff

[clang-tools-extra] [clangd] Augment code completion results with documentation from the index. (PR #120099)

https://github.com/HighCommander4 closed https://github.com/llvm/llvm-project/pull/120099 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clangd] Augment code completion results with documentation from the index. (PR #120099)

HighCommander4 wrote: > Seems to be template-related. > > test.hpp: > > ```c++ > template > struct templated { > // comment > void foo(); > }; > struct nontemplated { > // comment > void foo(); > }; > ``` > > test.cpp: > > ```c++ > #include "main.hpp" > void foo() { > nontemplated

[clang] [clang-format] Support BraceWrapping.AfterNamespace with AllowShortNamespacesOnASingleLine (PR #123010)

https://github.com/galenelias created https://github.com/llvm/llvm-project/pull/123010 AllowShortNamespacesOnASingleLine assumes that there is no newline before the namespace brace, however, there is no actual reason this shouldn't be compatible with BraceWrapping.AfterNamespace = true. This

[clang] [Sema] Migrate away from PointerUnion::dyn_cast (NFC) (PR #123014)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) Changes Note that PointerUnion::dyn_cast has been soft deprecated in PointerUnion.h: // FIXME: Replace the uses of is(), get() and dyn_cast() with //isa, cast and the llvm::dyn_cast Litera

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

@@ -196,3 +196,13 @@ Options // The following pointer may not become a 'int *const'. int *changing_pointee = &value; changing_pointee = &result; + +.. option:: AllowedTypes (default = '') + + A semicolon-separated list of names of types that + will be excluded fro

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

@@ -196,3 +196,13 @@ Options // The following pointer may not become a 'int *const'. int *changing_pointee = &value; changing_pointee = &result; + +.. option:: AllowedTypes (default = '') + + A semicolon-separated list of names of types that + will be excluded fro

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

@@ -196,3 +196,13 @@ Options // The following pointer may not become a 'int *const'. int *changing_pointee = &value; changing_pointee = &result; + +.. option:: AllowedTypes (default = '') + + A semicolon-separated list of names of types that + will be excluded fro

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

https://github.com/vbvictor updated https://github.com/llvm/llvm-project/pull/122951 >From 841cfec5a0ab4ce5ce64e71facfb7becaf4340e8 Mon Sep 17 00:00:00 2001 From: Victor Baranov Date: Tue, 14 Jan 2025 22:05:43 +0300 Subject: [PATCH 1/2] [clang-tidy] add AllowedTypes to misc-const-correctness -

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

@@ -45,6 +45,8 @@ class ConstCorrectnessCheck : public ClangTidyCheck { const bool TransformValues; const bool TransformReferences; const bool TransformPointersAsValues; + vbvictor wrote: Done https://github.com/llvm/llvm-project/pull/122951 ___

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

@@ -57,6 +61,9 @@ void ConstCorrectnessCheck::storeOptions(ClangTidyOptions::OptionMap &Opts) { Options.store(Opts, "TransformValues", TransformValues); Options.store(Opts, "TransformReferences", TransformReferences); Options.store(Opts, "TransformPointersAsValues", Tran

[clang] [Driver][ARM] Fix hardcoding of NEON FPU for armv8 (PR #122969)

https://github.com/Sharjeel-Khan edited https://github.com/llvm/llvm-project/pull/122969 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [lldb] [C++20] [Modules] Support module level lookup (PR #122887)

https://github.com/ChuanqiXu9 updated https://github.com/llvm/llvm-project/pull/122887 >From a259ccafe9b6f7efc740cd7203d44fd4170a25dc Mon Sep 17 00:00:00 2001 From: Chuanqi Xu Date: Thu, 26 Dec 2024 16:00:51 +0800 Subject: [PATCH] [C++20] [Modules] Support module level lookup Close https://git

[clang] [AST] Migrate away from PointerUnion::dyn_cast (NFC) (PR #123012)

https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/123012 Note that PointerUnion::dyn_cast has been soft deprecated in PointerUnion.h: // FIXME: Replace the uses of is(), get() and dyn_cast() with //isa, cast and the llvm::dyn_cast Literal migration

[clang] [AST] Migrate away from PointerUnion::dyn_cast (NFC) (PR #123012)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) Changes Note that PointerUnion::dyn_cast has been soft deprecated in PointerUnion.h: // FIXME: Replace the uses of is(), get() and dyn_cast() with //isa, cast and the llvm::dyn_cast Litera

[clang] [CodeGen] Migrate away from PointerUnion::dyn_cast (NFC) (PR #123013)

https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/123013 Note that PointerUnion::dyn_cast has been soft deprecated in PointerUnion.h: // FIXME: Replace the uses of is(), get() and dyn_cast() with //isa, cast and the llvm::dyn_cast Literal migration

[clang-tools-extra] [clang-tidy] add AllowedTypes option to misc-const-correctness (PR #122951)

@@ -196,3 +196,13 @@ Options // The following pointer may not become a 'int *const'. int *changing_pointee = &value; changing_pointee = &result; + +.. option:: AllowedTypes (default = '') + + A semicolon-separated list of names of types that + will be excluded fro

[clang] [Driver][ARM] Fix hardcoding of NEON FPU for armv8 (PR #122969)

https://github.com/Sharjeel-Khan updated https://github.com/llvm/llvm-project/pull/122969 >From ebbb016b161165fff8335213e89dbbf8135a5329 Mon Sep 17 00:00:00 2001 From: Sharjeel Khan Date: Tue, 14 Jan 2025 19:02:31 + Subject: [PATCH] Fix hardcoding of NEON FPU for armv8 Android hardcoded NE

[clang] [CodeGen] Migrate away from PointerUnion::dyn_cast (NFC) (PR #123013)

llvmbot wrote: @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) Changes Note that PointerUnion::dyn_cast has been soft deprecated in PointerUnion.h: // FIXME: Replace the uses of is(), get() and dyn_cast() with //isa, cast and the llvm::dyn_cast Litera

[clang] [CodeGen] Migrate away from PointerUnion::dyn_cast (NFC) (PR #123013)

llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Kazu Hirata (kazutakahirata) Changes Note that PointerUnion::dyn_cast has been soft deprecated in PointerUnion.h: // FIXME: Replace the uses of is(), get() and dyn_cast() with //isa, cast and the llvm::dyn_cast

[clang] [Sema] Migrate away from PointerUnion::dyn_cast (NFC) (PR #123014)

https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/123014 Note that PointerUnion::dyn_cast has been soft deprecated in PointerUnion.h: // FIXME: Replace the uses of is(), get() and dyn_cast() with //isa, cast and the llvm::dyn_cast Literal migration

<    1   2   3   4   5   6   >