https://github.com/MacDue edited
https://github.com/llvm/llvm-project/pull/124590
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Pedro Lobo (pedroclobo)
Changes
Merge builtins with merge type `MergeAnyExp` with `poison` operands instead of
`undef` operands.
---
Patch is 270.55 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/p
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Pedro Lobo (pedroclobo)
Changes
Merge builtins with merge type `MergeAnyExp` with `poison` operands instead of
`undef` operands.
---
Patch is 270.55 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-p
@@ -31,6 +31,6 @@ Options
A semicolon-separated list of names of types allowed to be copied in each
iteration. Regular expressions are accepted, e.g. `[Rr]ef(erence)?$` matches
every type with suffix `Ref`, `ref`, `Reference` and `reference`. The
default
-
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/121394
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/TilakChad edited
https://github.com/llvm/llvm-project/pull/124609
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2,10 +2,10 @@
// UNSUPPORTED: target=thumb{{.*}}
// UNSUPPORTED: android
-// RUN: %clangxx -fsanitize=return %gmlt -O2 -fno-omit-frame-pointer
-fasynchronous-unwind-tables %s -o %t
+// RUN: %clangxx -Wno-error=return-type -fsanitize=return %gmlt -O2
-fno-omit-frame-point
https://github.com/kuilpd updated
https://github.com/llvm/llvm-project/pull/115005
>From 4d797371598960baf7729d05590aa1a8c7077694 Mon Sep 17 00:00:00 2001
From: Ilia Kuklin
Date: Mon, 4 Nov 2024 14:33:45 +0500
Subject: [PATCH 1/8] [lldb] Analyze enum promotion type during parsing
---
clang/in
@@ -875,7 +875,41 @@ void TextNodeDumper::dumpBareDeclRef(const Decl *D) {
if (const NamedDecl *ND = dyn_cast(D)) {
ColorScope Color(OS, ShowColors, DeclNameColor);
-OS << " '" << ND->getDeclName() << '\'';
+if (DeclarationName Name = ND->getDeclName())
+ OS
@@ -2,10 +2,10 @@
// UNSUPPORTED: target=thumb{{.*}}
// UNSUPPORTED: android
-// RUN: %clangxx -fsanitize=return %gmlt -O2 -fno-omit-frame-pointer
-fasynchronous-unwind-tables %s -o %t
+// RUN: %clangxx -Wno-error=return-type -fsanitize=return %gmlt -O2
-fno-omit-frame-point
@@ -1,4 +1,4 @@
-// RUN: %clang_hwasan %s -o %t && %env_hwasan_opts=random_tags=1 %run %t
+// RUN: %clang_hwasan -Wno-error=return-type %s -o %t &&
%env_hwasan_opts=random_tags=1 %run %t
Sirraide wrote:
I’m not really that familiar with the sanitizers, and I kno
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: TilakChad (TilakChad)
Changes
The `UnresolvedLookupExpr` doesn't get looked up and resolved again while it is
in the non-dependent context. It propagates into the codegen phase and causing
the assertion failure.
We attempt to determine i
https://github.com/Sirraide edited
https://github.com/llvm/llvm-project/pull/123470
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -875,7 +875,41 @@ void TextNodeDumper::dumpBareDeclRef(const Decl *D) {
if (const NamedDecl *ND = dyn_cast(D)) {
ColorScope Color(OS, ShowColors, DeclNameColor);
-OS << " '" << ND->getDeclName() << '\'';
+if (DeclarationName Name = ND->getDeclName())
+ OS
Sirraide wrote:
Aaron and I just talked about this a bit, and I agree it would be cleaner to
just make a single function that can be used everywhere and whose behaviour is
customised with an enum. I’ll experiment with that approach and see how it
goes. Also, my plan was to try and remove `isPo
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-libc-amdgpu-runtime` running on `omp-vega20-1` while building
`clang` at step 7 "Add check check-offload".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/73/builds/12500
Here is the r
https://github.com/TilakChad created
https://github.com/llvm/llvm-project/pull/124609
The `UnresolvedLookupExpr` doesn't get looked up and resolved again while it is
in the non-dependent context. It propagates into the codegen phase and causing
the assertion failure.
We attempt to determine i
@@ -0,0 +1,39 @@
+.. title:: clang-tidy - use-cpp-style-comments
+
+readability-use-cpp-style-comments
+==
+
+Replace C-style comments with C++-style comments.
+C-style comments (``/* ... */``) are inherited from C, while C++ introduces
+``//`` as
@@ -1228,6 +1228,21 @@ def HasVendorXCVbi
: Predicate<"Subtarget->hasVendorXCVbi()">,
AssemblerPredicate<(all_of FeatureVendorXCVbi),
"'XCVbi' (CORE-V Immediate Branching)">;
+// MIPS Extensions
+
+def FeatureVendorXMIPSCMove
+: RISCVExten
@@ -0,0 +1,39 @@
+.. title:: clang-tidy - use-cpp-style-comments
+
+readability-use-cpp-style-comments
+==
+
+Replace C-style comments with C++-style comments.
+C-style comments (``/* ... */``) are inherited from C, while C++ introduces
+``//`` as
@@ -173,13 +173,18 @@ New checks
Replace comparisons between signed and unsigned integers with their safe
C++20 ``std::cmp_*`` alternative, if available.
-
+
EugeneZelenko wrote:
Please remove spaces.
https://github.com/llvm/llvm-project/pull/124319
__
https://github.com/4m4n-x-B4w4ne updated
https://github.com/llvm/llvm-project/pull/124319
>From 581cc0eda9d42458ff71f2c913a2e03d2de0b5f2 Mon Sep 17 00:00:00 2001
From: NagaChaitanya Vellanki
Date: Fri, 19 Jul 2024 14:26:23 -0700
Subject: [PATCH 01/12] This commit is to resolve the issue #24841
https://github.com/nickdesaulniers approved this pull request.
Thanks, I appreciate the comments about common usage patterns; they help make
it easier to understand (subjectively) the 4 cases you handle in code. I still
think we may need to triple check how this extension behaves with count fi
@@ -514,9 +514,7 @@ available in all language modes.
__nullptr
-
-``__nullptr`` is an alternate spelling for ``nullptr``, but is also available
in
-C++ modes prior to C++11. Note that it's currently not availbale in C despite
-C23 having support for ``nullptr``.
+``__
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/123119
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,4 +1,4 @@
-// RUN: %clang_hwasan %s -o %t && %env_hwasan_opts=random_tags=1 %run %t
+// RUN: %clang_hwasan -Wno-error=return-type %s -o %t &&
%env_hwasan_opts=random_tags=1 %run %t
vitalybuka wrote:
It should be fine to change HWASAN tests, it does nothing
https://github.com/AaronBallman approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/122486
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
bwendling wrote:
> I still think we may need to triple check how this extension behaves with
> count fields that do not get extended AND are negative at runtime (if we
> don't already have tests for that), but perhaps is orthogonal to this initial
> refactoring.
I'll look into test cases with
rnk wrote:
+1 to separating implementation from enablement. Enabling the warning by
default will predictably generate pushback and folks may ask to revert, and
it's a good best practice to be able to easily do that while leaving the
implementation in place.
https://github.com/llvm/llvm-projec
https://github.com/glebov-andrey updated
https://github.com/llvm/llvm-project/pull/114075
>From 7c4f07ddbc8cdb77e8526dccd6bd19fa5a64744a Mon Sep 17 00:00:00 2001
From: Andrey Glebov
Date: Mon, 28 Oct 2024 00:02:51 +0300
Subject: [PATCH] [clang] MicrosoftCXXABI: restore the RecordToCopyCtor tabl
https://github.com/AidanGoldfarb updated
https://github.com/llvm/llvm-project/pull/123119
>From bc5d1419d94ac492c0fcf62a74844eb43ce17f1a Mon Sep 17 00:00:00 2001
From: Aidan
Date: Wed, 15 Jan 2025 15:24:12 -0500
Subject: [PATCH 01/13] __nullptr -> KEYALL, added relevant test
---
clang/include
Sirraide wrote:
ping
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
Sirraide wrote:
ping
https://github.com/llvm/llvm-project/pull/122991
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -196,3 +196,12 @@ Options
// The following pointer may not become a 'int *const'.
int *changing_pointee = &value;
changing_pointee = &result;
+
+.. option:: AllowedTypes (default = '')
EugeneZelenko wrote:
Defaults usually stated in last sentenc
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/118409
>From 86bc2d2fe03d84879870eec1a8c10912076686c2 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Mon, 2 Dec 2024 15:16:39 -0700
Subject: [PATCH 1/4] [clang-format] Add PenaltyBreakBeforeMemberAccess
Add a config
https://github.com/frasercrmck created
https://github.com/llvm/llvm-project/pull/124568
By using the vector reduction buitins we can avoid scalarization. Targets that
don't support vector reductions will scalarize later on anyway. The vector
reduction builtins should be well-enough supported b
@@ -196,3 +196,12 @@ Options
// The following pointer may not become a 'int *const'.
int *changing_pointee = &value;
changing_pointee = &result;
+
+.. option:: AllowedTypes (default = '')
vbvictor wrote:
In this file all the options has default in
balazske wrote:
It seems that this test is not reducible any further (except very small
changes) (to get the original crash). I could remove only few lines.
https://github.com/llvm/llvm-project/pull/124273
___
cfe-commits mailing list
cfe-commits@list
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Younan Zhang (zyn0217)
Changes
This is taken from
https://github.com/llvm/llvm-project/pull/122423#discussion_r1930102632 as
suggested by Corentin.
Fixes #82926
---
Full diff: https://github.com/llvm/llvm-project/pull/124572.diff
3 Fi
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/7] [clang-tidy] add AllowedTypes to misc-const-correctness
-
@@ -24,11 +24,24 @@ namespace offloading {
/// This is the record of an object that just be registered with the offloading
/// runtime.
struct EntryTy {
+ /// Reserved bytes used to detect an older version of the struct, always
zero.
+ uint64_t Reserved = 0x0;
--
Author: Fraser Cormack
Date: 2025-01-27T16:37:21Z
New Revision: c3a0fcc982061f9a69cfc1199dc91bd1fc3158c0
URL:
https://github.com/llvm/llvm-project/commit/c3a0fcc982061f9a69cfc1199dc91bd1fc3158c0
DIFF:
https://github.com/llvm/llvm-project/commit/c3a0fcc982061f9a69cfc1199dc91bd1fc3158c0.diff
LOG
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/124364
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Jon Roelofs
Date: 2025-01-27T08:49:32-08:00
New Revision: 73db9ee1e87b4cfccbc9d67d2b47d9476f92413f
URL:
https://github.com/llvm/llvm-project/commit/73db9ee1e87b4cfccbc9d67d2b47d9476f92413f
DIFF:
https://github.com/llvm/llvm-project/commit/73db9ee1e87b4cfccbc9d67d2b47d9476f92413f.diff
L
@@ -0,0 +1,127 @@
+//===--- UseCppStyleCommentsCheck.cpp -
clang-tidy-===//
+
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
@@ -70,6 +70,10 @@ def err_drv_no_rocm_device_lib : Error<
"cannot find ROCm device library%select{| for %1| for ABI version %1}0;
provide its path via "
"'--rocm-path' or '--rocm-device-lib-path', or pass '-nogpulib' to build "
"without ROCm device library">;
+def err_d
https://github.com/shafik commented:
In my comment here, it really looks like a clang-17 regression:
https://github.com/llvm/llvm-project/issues/124161#event-16079402729
>From what I can tell you also address the simplified version, so I believe a
>release note is probably warrented.
Are ther
@@ -0,0 +1,127 @@
+//===--- UseCppStyleCommentsCheck.cpp -
clang-tidy-===//
+
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
@@ -0,0 +1,40 @@
+.. title:: clang-tidy - use-cpp-style-comments
+
+modernize-use-cpp-style-comments
+
+
+Replace C-style comments with C++-style comments.
4m4n-x-B4w4ne wrote:
Done Thanks.
https://github.com/llvm/llvm-project/pul
https://github.com/thebrandre edited
https://github.com/llvm/llvm-project/pull/124407
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,127 @@
+//===--- UseCppStyleCommentsCheck.cpp -
clang-tidy-===//
+
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
@@ -0,0 +1,127 @@
+//===--- UseCppStyleCommentsCheck.cpp -
clang-tidy-===//
+
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier:
@@ -0,0 +1,40 @@
+.. title:: clang-tidy - use-cpp-style-comments
+
+modernize-use-cpp-style-comments
+
+
+Replace C-style comments with C++-style comments.
+C-style comments (``/* ... */``) are inherited from C, while C++ introduces
+``//`` as a mo
@@ -0,0 +1,68 @@
+// RUN: %check_clang_tidy -std=c++11 %s modernize-use-cpp-style-comments %t --
--
+
+// Single-line full C-style comment
+static const int CONSTANT = 42; /* Important constant value */
+// CHECK-MESSAGES: :[[@LINE-1]]:33: warning: use C++ style comments '//'
in
@@ -0,0 +1,40 @@
+//===--- UseCppStyleCommentsCheck.h -
clang-tidy---===//
4m4n-x-B4w4ne wrote:
Done Thanks.
https://github.com/llvm/llvm-project/pull/124319
___
cfe-commits mailing list
cfe-co
cor3ntin wrote:
I'll push a fix for the build failure on arm shortly
https://github.com/llvm/llvm-project/pull/124404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/4m4n-x-B4w4ne deleted
https://github.com/llvm/llvm-project/pull/124319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
4m4n-x-B4w4ne wrote:
Can anyone please help me? Why is Test Documentation build is failing? I am in
no clue.
https://github.com/llvm/llvm-project/pull/124319
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailma
https://github.com/anematode updated
https://github.com/llvm/llvm-project/pull/124365
>From 2508da6fd7f38101011573460724f13df1c1616d Mon Sep 17 00:00:00 2001
From: Timothy Herchen
Date: Fri, 24 Jan 2025 15:13:30 -0800
Subject: [PATCH] [clang][docs] Fix typos concerning wasm __funcref
---
clan
https://github.com/4m4n-x-B4w4ne updated
https://github.com/llvm/llvm-project/pull/124319
>From 581cc0eda9d42458ff71f2c913a2e03d2de0b5f2 Mon Sep 17 00:00:00 2001
From: NagaChaitanya Vellanki
Date: Fri, 19 Jul 2024 14:26:23 -0700
Subject: [PATCH 1/8] This commit is to resolve the issue #24841
-
https://github.com/MacDue created
https://github.com/llvm/llvm-project/pull/124590
Currently, these generate incorrect code, as streaming attributes are not
propagated to the outlined function. As we've yet to work on mixing OpenMP and
streaming functions (and determine how they should interac
AaronBallman wrote:
> In my comment here, it really looks like a clang-17 regression: [#124161
> (comment)](https://github.com/llvm/llvm-project/issues/124161#event-16079402729)
>
> From what I can tell you also address the simplified version, so I believe a
> release note is probably warrente
@@ -1211,6 +1211,15 @@ WebAssemblyTargetLowering::LowerCall(CallLoweringInfo
&CLI,
}
}
}
+
+// If outgoing arguments are passed via the stack, we cannot tail call
+for (const ISD::OutputArg &Out : CLI.Outs) {
anematode wrote:
Whoops!
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Benjamin Maxwell (MacDue)
Changes
Currently, these generate incorrect code, as streaming attributes are not
propagated to the outlined function. As we've yet to work on mixing OpenMP and
streaming functions (and determine how they should
@@ -4568,9 +4568,23 @@ buildCapturedStmtCaptureList(Sema &S,
CapturedRegionScopeInfo *RSI,
return false;
}
+static bool
+isOpenMPCapturedRegionInArmStreamingFunction(Sema const &S,
+ CapturedRegionKind Kind) {
+ if (!S.getLangOpt
https://github.com/AidanGoldfarb updated
https://github.com/llvm/llvm-project/pull/123119
>From bc5d1419d94ac492c0fcf62a74844eb43ce17f1a Mon Sep 17 00:00:00 2001
From: Aidan
Date: Wed, 15 Jan 2025 15:24:12 -0500
Subject: [PATCH 1/9] __nullptr -> KEYALL, added relevant test
---
clang/include/c
@@ -1228,6 +1228,21 @@ def HasVendorXCVbi
: Predicate<"Subtarget->hasVendorXCVbi()">,
AssemblerPredicate<(all_of FeatureVendorXCVbi),
"'XCVbi' (CORE-V Immediate Branching)">;
+// MIPS Extensions
+
+def FeatureVendorXMIPSCMove
+: RISCVExten
@@ -1228,6 +1228,21 @@ def HasVendorXCVbi
: Predicate<"Subtarget->hasVendorXCVbi()">,
AssemblerPredicate<(all_of FeatureVendorXCVbi),
"'XCVbi' (CORE-V Immediate Branching)">;
+// MIPS Extensions
+
+def FeatureVendorXMIPSCMove
+: RISCVExten
https://github.com/frasercrmck edited
https://github.com/llvm/llvm-project/pull/124598
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AidanGoldfarb updated
https://github.com/llvm/llvm-project/pull/123119
>From bc5d1419d94ac492c0fcf62a74844eb43ce17f1a Mon Sep 17 00:00:00 2001
From: Aidan
Date: Wed, 15 Jan 2025 15:24:12 -0500
Subject: [PATCH 01/10] __nullptr -> KEYALL, added relevant test
---
clang/include
https://github.com/lenary approved this pull request.
Thanks!
https://github.com/llvm/llvm-project/pull/124237
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3903,6 +3903,7 @@ class EnumDecl : public TagDecl {
void setInstantiationOfMemberEnum(ASTContext &C, EnumDecl *ED,
TemplateSpecializationKind TSK);
+public:
kuilpd wrote:
I left this in so that that I could simply do
https://github.com/mizvekov updated
https://github.com/llvm/llvm-project/pull/124605
>From cf6413e220574adac3c89b28cc5011050ce11144 Mon Sep 17 00:00:00 2001
From: Matheus Izvekov
Date: Mon, 27 Jan 2025 15:08:01 -0300
Subject: [PATCH 1/2] NFC: convert
clang/test/AST/HLSL/StructuredBuffers-AST.h
https://github.com/AaronBallman approved this pull request.
Thank you for this cleanup -- LGTM! Thank you for investigating the compile
time overhead impacts ahead of time, that's appreciated. :-)
https://github.com/llvm/llvm-project/pull/122991
___
c
https://github.com/AaronBallman approved this pull request.
LGTM aside from a change to the documentation. Thank you!
https://github.com/llvm/llvm-project/pull/123119
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
rjodinchr wrote:
@frasercrmck, could you review please?
https://github.com/llvm/llvm-project/pull/124614
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/124605
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/118409
>From 86bc2d2fe03d84879870eec1a8c10912076686c2 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Mon, 2 Dec 2024 15:16:39 -0700
Subject: [PATCH 1/7] [clang-format] Add PenaltyBreakBeforeMemberAccess
Add a config
@@ -104,9 +104,39 @@ class PrototypeParser {
void ParseType(StringRef T) {
T = T.trim();
+
+auto ConsumeAddrSpace = [&]() -> std::optional {
+ T = T.trim();
+ if (!T.consume_back(">"))
+return std::nullopt;
+
+ auto Open = T.find_last_of('<');
@@ -31,6 +31,6 @@ Options
A semicolon-separated list of names of types allowed to be copied in each
iteration. Regular expressions are accepted, e.g. `[Rr]ef(erence)?$` matches
every type with suffix `Ref`, `ref`, `Reference` and `reference`. The
default
-
AaronBallman wrote:
> Ultimately we want
>
> 1. I know all of these attributes, don't tell me about them
>
> 2. I know all of these attribute namespaces, don't tell me about them -
> which this patch is doing
>
> 3. Have a way to disable unknown attributes warnings for all unknown
@@ -875,7 +875,41 @@ void TextNodeDumper::dumpBareDeclRef(const Decl *D) {
if (const NamedDecl *ND = dyn_cast(D)) {
ColorScope Color(OS, ShowColors, DeclNameColor);
-OS << " '" << ND->getDeclName() << '\'';
+if (DeclarationName Name = ND->getDeclName())
+ OS
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/118409
>From 86bc2d2fe03d84879870eec1a8c10912076686c2 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Mon, 2 Dec 2024 15:16:39 -0700
Subject: [PATCH 1/6] [clang-format] Add PenaltyBreakBeforeMemberAccess
Add a config
https://github.com/rjodinchr created
https://github.com/llvm/llvm-project/pull/124614
clc_isnan.cl is needed since https://github.com/llvm/llvm-project/pull/124097
>From 1c11070d2f2997b76412e7be07fa05501147c5ff Mon Sep 17 00:00:00 2001
From: Romaric Jodin
Date: Mon, 27 Jan 2025 20:21:31 +0100
https://github.com/Artem-B approved this pull request.
LGTM overall with a couple of nits.
I like the direction of the change. Tablegen is probably a better way to handle
NVPTX quirks, than a preprocessor. We may finally be able to replace the
string-based constraints that are growing a bit to
@@ -0,0 +1,1078 @@
+//===--- BuiltinsNVPTX.td - NVPTX Builtin function defs -*- C++
-*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: A
https://github.com/Artem-B edited
https://github.com/llvm/llvm-project/pull/122873
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
nikic wrote:
FYI this change has some measurable compile-time overhead (about 0.2% for debug
builds):
https://llvm-compile-time-tracker.com/compare.php?from=eaa5897534cbd263d0cdbf780f72133c2fe8d8d4&to=561132e71b29d9b747dfda1509f715847852f77b&stat=instructions:u
Not sure whether that's expected
https://github.com/Acim-Maravic created
https://github.com/llvm/llvm-project/pull/124616
None
>From dea0bd7a2cb5f67599257a7d6ae6d0bb0e8804ab Mon Sep 17 00:00:00 2001
From: Acim Maravic
Date: Mon, 27 Jan 2025 19:26:03 +0100
Subject: [PATCH] [LLVM][AMDGPU] Add Intrinsic and Builtin for
ds_bperm
llvmbot wrote:
@llvm/pr-subscribers-llvm-ir
Author: Acim Maravic (Acim-Maravic)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/124616.diff
6 Files Affected:
- (modified) clang/include/clang/Basic/BuiltinsAMDGPU.def (+2)
- (modified) clang/test/CodeGenOpenCL/builtins
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Acim Maravic (Acim-Maravic)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/124616.diff
6 Files Affected:
- (modified) clang/include/clang/Basic/BuiltinsAMDGPU.def (+2)
- (modified) clang/test/CodeGenOpenCL/builtins-a
@@ -11682,6 +11683,7 @@ class Sema final : public SemaBase {
SmallVectorImpl &SugaredConverted,
SmallVectorImpl &CanonicalConverted,
CheckTemplateArgumentKind CTAK, bool PartialOrdering,
+
@@ -11682,6 +11683,7 @@ class Sema final : public SemaBase {
SmallVectorImpl &SugaredConverted,
SmallVectorImpl &CanonicalConverted,
CheckTemplateArgumentKind CTAK, bool PartialOrdering,
+
vbvictor wrote:
As @PiotrZSL mentioned in the issue, adding an option called
`ExcludeDoxygenStyleComments` would be very helpful (or even necessary) before
releasing this check. Otherwise, it could create a lot of trouble for projects
that use doxygen for their documentation (including LLVM it
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
@@ -31,6 +31,6 @@ Options
A semicolon-separated list of names of types allowed to be copied in each
iteration. Regular expressions are accepted, e.g. `[Rr]ef(erence)?$` matches
every type with suffix `Ref`, `ref`, `Reference` and `reference`. The
default
-
efriedma-quic wrote:
See also discussion on #117845. I think it makes some sense to add a way for
CodeGen to query whether a given declaration is impossible to emit. I
suggested in that discussion that it could be a kind of linkage.
I'd prefer not to mess with the way DeferredDecls works for
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
https://github.com/shafik commented:
I second that we need to document how values outside of zero and one are
handled.
I would like also really want to see a test that demonstrates the behavior we
are expecting to support with comments explaining the result and why we expect
it.
After seei
apple-fcloutier wrote:
I will get back to this later.
https://github.com/llvm/llvm-project/pull/116732
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
401 - 500 of 597 matches
Mail list logo
