Author: Owen Pan
Date: 2025-07-06T23:54:00-07:00
New Revision: a5af8745039f906c4fc4184a1fe0d35d42355f52
URL:
https://github.com/llvm/llvm-project/commit/a5af8745039f906c4fc4184a1fe0d35d42355f52
DIFF:
https://github.com/llvm/llvm-project/commit/a5af8745039f906c4fc4184a1fe0d35d42355f52.diff
LOG:
@@ -2942,6 +2942,65 @@ void tools::addHIPRuntimeLibArgs(const ToolChain &TC,
Compilation &C,
}
}
+void tools::addOpenCLBuiltinsLib(const Driver &D,
+ const llvm::opt::ArgList &DriverArgs,
+ llvm::opt::ArgString
@@ -2942,6 +2942,65 @@ void tools::addHIPRuntimeLibArgs(const ToolChain &TC,
Compilation &C,
}
}
+void tools::addOpenCLBuiltinsLib(const Driver &D,
+ const llvm::opt::ArgList &DriverArgs,
+ llvm::opt::ArgString
martinboehme wrote:
As an initial comment, wanted to confirm that we've clarified what the
semantics of the AST are here.
In https://github.com/llvm/llvm-project/issues/128068, I asked:
> What I'm not clear on is what the rule is that disambiguates this. Is it
> simply that if a `CXXThisExpr`
owenca wrote:
Had we used `StringRef::contains` initially, would you still insist that it be
replaced with a "more performant" local function? Please note that the code is
not on a performance critical path, and the difference in time it takes to run
the relevant unit tests for 100 times is wi
@@ -0,0 +1,8 @@
+// RUN: %clang_analyze_cc1 -std=c++23 -analyzer-checker=core -verify %s
imdj wrote:
> Please check every standard from C++23 and up
What is the best approach to do this? I noticed in some files they duplicate
the line for each subsequent standa
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp --
clang-tools-extra/clang-tidy/bugprone/UnhandledSelfAs
https://github.com/topperc approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/146309
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,62 @@
+.. title:: clang-tidy - misc-bool-bitwise-operation
+
+misc-bool-bitwise-operation
+===
+
+Finds potentially inefficient use of bitwise operators such as ``&``, ``|``
+and their compound analogues on Boolean values where logical operator
https://github.com/tclin914 closed
https://github.com/llvm/llvm-project/pull/147005
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Jim Lin
Date: 2025-07-07T13:01:22+08:00
New Revision: 3f33e7ba5b08a97939280e539ad20a34d54b2719
URL:
https://github.com/llvm/llvm-project/commit/3f33e7ba5b08a97939280e539ad20a34d54b2719
DIFF:
https://github.com/llvm/llvm-project/commit/3f33e7ba5b08a97939280e539ad20a34d54b2719.diff
LOG:
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-aarch64-windows`
running on `linaro-armv8-windows-msvc-05` while building `clang` at step 6
"test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/141/builds/9984
Here is the relevant piece of
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-x86_64-win` running
on `as-builder-10` while building `clang` at step 8 "test-check-lldb-unit".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/211/builds/284
Here is the relevant piece of the bu
https://github.com/madhur13490 closed
https://github.com/llvm/llvm-project/pull/147014
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/147203
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2025-07-06T21:06:33-07:00
New Revision: 9d8a1bec2b55dfc21b84202bb0d1a4e04c8470e8
URL:
https://github.com/llvm/llvm-project/commit/9d8a1bec2b55dfc21b84202bb0d1a4e04c8470e8
DIFF:
https://github.com/llvm/llvm-project/commit/9d8a1bec2b55dfc21b84202bb0d1a4e04c8470e8.diff
L
@@ -0,0 +1,62 @@
+.. title:: clang-tidy - misc-bool-bitwise-operation
+
+misc-bool-bitwise-operation
+===
+
+Finds potentially inefficient use of bitwise operators such as ``&``, ``|``
+and their compound analogues on Boolean values where logical operator
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/147231
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yxsamliu updated
https://github.com/llvm/llvm-project/pull/145131
>From 0492b8d9db17c8a59390938033b6d6c8d12946e1 Mon Sep 17 00:00:00 2001
From: "Yaxun (Sam) Liu"
Date: Tue, 17 Jun 2025 17:01:07 -0400
Subject: [PATCH] [Clang][Driver] Add jobserver support for --offload-jobs
T
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/147203
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/YexuanXiao edited
https://github.com/llvm/llvm-project/pull/143653
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jj-marr wrote:
> Please remove unnecessary comments that tell the same as code itself,
> speaking mostly about comments in `registerMatchers`. They mostly don't give
> any more information that matchers themselves.
To be honest, this is because I coded 98% of this with AI (except for the
`.rs
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Bogdan Vetrenko (bv2k4)
Changes
This change replaces manual `new[]`/`delete[]` with `llvm::SmallVector` for
`TemplateArgumentLocInfo` in `createTrivialConceptReference`.
---
Full diff: https://github.com/llvm/llvm-project/pull/147231.diff
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
https://github.com/bv2k4 created
https://github.com/llvm/llvm-project/pull/147231
This change replaces manual `new[]`/`delete[]` with `llvm::SmallVector` for
`TemplateArgumentLocInfo` in `createTrivialConceptReference`.
>From fd77cdcc7e0a1e55ddcbf74fa86517623287d974 Mon Sep 17 00:00:00 2001
Fr
denzor200 wrote:
> we could add Limitations section in check docs to avoid potential confusion
> and false-negative reports in the future.
Done
https://github.com/llvm/llvm-project/pull/142324
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
h
@@ -0,0 +1,260 @@
+//===--- AvoidPlatformSpecificFundamentalTypesCheck.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: Ap
@@ -1432,6 +1442,26 @@ static bool runImpl(Module &M, AnalysisGetter &AG,
TargetMachine &TM,
} else if (auto *CmpX = dyn_cast(&I)) {
A.getOrCreateAAFor(
IRPosition::value(*CmpX->getPointerOperand()));
+ } else if (auto *II = dyn_cast(&I)) {
+
@@ -5501,7 +5505,32 @@ struct AAAlignCallSiteReturned final
using Base = AACalleeToCallSite;
AAAlignCallSiteReturned(const IRPosition &IRP, Attributor &A)
: Base(IRP, A) {}
+ ChangeStatus updateImpl(Attributor &A) override {
+SmallVector Values;
+SmallVector
https://github.com/wenju-he updated
https://github.com/llvm/llvm-project/pull/146814
>From 6429b5538dcfcd9de262f6f0a49d8283db8f5db9 Mon Sep 17 00:00:00 2001
From: Wenju He
Date: Wed, 2 Jul 2025 17:11:56 -0700
Subject: [PATCH 1/2] [libclc] Add generic implementation of some atomic
functions in
@@ -85,7 +85,7 @@ __amdgpu_buffer_rsrc_t
test_amdgcn_make_buffer_p0_nullptr(short stride, int num,
// CHECK-LABEL: @test_amdgcn_make_buffer_p1_nullptr(
// CHECK-NEXT: entry:
-// CHECK-NEXT:[[TMP0:%.*]] = tail call ptr addrspace(8)
@llvm.amdgcn.make.buffer.rsrc.p8.p1(ptr
https://github.com/Shoreshen updated
https://github.com/llvm/llvm-project/pull/145278
>From 888df5412b37bd3f232bdb38c9f89786d042fe75 Mon Sep 17 00:00:00 2001
From: shore <372660...@qq.com>
Date: Mon, 23 Jun 2025 14:12:15 +0800
Subject: [PATCH 1/5] Add alignment attr & propagate alignment through
Author: Kazu Hirata
Date: 2025-07-06T19:05:52-07:00
New Revision: 15653108b11ebd29004f2fba44a5c757b8a7efa8
URL:
https://github.com/llvm/llvm-project/commit/15653108b11ebd29004f2fba44a5c757b8a7efa8
DIFF:
https://github.com/llvm/llvm-project/commit/15653108b11ebd29004f2fba44a5c757b8a7efa8.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/147204
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kazutakahirata edited
https://github.com/llvm/llvm-project/pull/147204
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 approved this pull request.
https://github.com/llvm/llvm-project/pull/147204
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 approved this pull request.
https://github.com/llvm/llvm-project/pull/147138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 approved this pull request.
LGTM. Thanks. A small suggestion is, it is better to paste the link of the
paper. So that I don't have to find it my self.
https://github.com/llvm/llvm-project/pull/147135
___
cfe-commits mail
https://github.com/ChuanqiXu9 approved this pull request.
https://github.com/llvm/llvm-project/pull/146993
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 approved this pull request.
https://github.com/llvm/llvm-project/pull/145220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/igorkudrin updated
https://github.com/llvm/llvm-project/pull/147221
>From f1e26fed6c5023ba59990112ec4a77b024247e4b Mon Sep 17 00:00:00 2001
From: Igor Kudrin
Date: Fri, 4 Jul 2025 23:55:41 -0700
Subject: [PATCH 1/2] [clang] Fix -Wuninitialized for values passed by const
poin
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/147204
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighW4y2H3ll updated
https://github.com/llvm/llvm-project/pull/145652
>From 9a29dd6fa3f28bf507f047a22597f8510bd096b3 Mon Sep 17 00:00:00 2001
From: h2h
Date: Tue, 24 Jun 2025 23:24:32 -0700
Subject: [PATCH 1/4] [Clang] Allow vanilla C function symbol name to be used
in __att
https://github.com/isuckatcs edited
https://github.com/llvm/llvm-project/pull/146859
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,8 @@
+// RUN: %clang_analyze_cc1 -std=c++23 -analyzer-checker=core -verify %s
isuckatcs wrote:
Please check every standard from C++23 and up. We want to ensure that this case
is handled properly for C++2, and every later standard as well.
https://git
@@ -0,0 +1,8 @@
+// RUN: %clang_analyze_cc1 -std=c++23 -analyzer-checker=core -verify %s
+
+void test_consteval() {
+ if consteval {
isuckatcs wrote:
Please add one more test for the `if !consteval` case too.
https://github.com/llvm/llvm-project/pull/146859
___
@@ -2794,6 +2794,9 @@ PathDiagnosticPieceRef
ConditionBRVisitor::VisitTerminator(
default:
return nullptr;
case Stmt::IfStmtClass:
+// Handle if consteval which doesn't have a traditional condition
+if (cast(Term)->isConsteval())
+ return nullptr;
Con
https://github.com/isuckatcs requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/146859
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kasuga-fj wrote:
As far as I can tell, it doesn't seem like a link to UsersManual is there, but
you can reach both from [the top page](https://clang.llvm.org/docs/index.html).
https://github.com/llvm/llvm-project/pull/147014
___
cfe-commits mailing li
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Author: Igor Kudrin (igorkudrin)
Changes
This enables producing a "variable is uninitialized" warning when a value is
passed to a pointer-to-const argument:
```
void foo(const int *);
void test() {
int *v;
foo(v);
}
```
Fixes #37460
https://github.com/igorkudrin created
https://github.com/llvm/llvm-project/pull/147221
This enables producing a "variable is uninitialized" warning when a value is
passed to a pointer-to-const argument:
```
void foo(const int *);
void test() {
int *v;
foo(v);
}
```
Fixes #37460
>From f1e
https://github.com/wenju-he closed
https://github.com/llvm/llvm-project/pull/146999
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shashi1687 updated
https://github.com/llvm/llvm-project/pull/147213
>From 05211dac479bf25aaf6d8ecc3a53871a51f7ffdd Mon Sep 17 00:00:00 2001
From: Shashi Shankar
Date: Sun, 6 Jul 2025 22:25:48 +0200
Subject: [PATCH] Sema: filter out invalid base-specifiers before attaching
Ac
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/147219
>From a5e5e795e2b31295f781d2398e41f46785c0f18b Mon Sep 17 00:00:00 2001
From: Oleksandr Tarasiuk
Date: Mon, 7 Jul 2025 02:52:01 +0300
Subject: [PATCH] [Clang] accept @tparam on variable template partial
spec
https://github.com/wenju-he approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/146840
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp --
clang/lib/AST/Comment.cpp clang/test/Sema/warn-docume
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Oleksandr T. (a-tarasyuk)
Changes
Fixes #144775
---
This patch addresses a false-positive `-Wdocumentation` warning on `@tparam` comments attached to _variable template partial specializations_
---
Full diff: https://github.com/llvm/llv
https://github.com/a-tarasyuk created
https://github.com/llvm/llvm-project/pull/147219
Fixes #144775
---
This patch addresses a false-positive `-Wdocumentation` warning on `@tparam`
comments attached to _variable template partial specializations_
>From 2c44c8bd3c2a49ba0a2cecdcadf5d08c0e9e31
https://github.com/clementval approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/146915
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kikairoya wrote:
I learned that a symlink works fine, so adding it to install-target might be a
good idea. I'll look into that later.
https://github.com/llvm/llvm-project/pull/147132
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
https://github.com/4vtomat updated
https://github.com/llvm/llvm-project/pull/145489
>From 88d1184bf6c8ac71c3a80a402e7e0122aad2bf52 Mon Sep 17 00:00:00 2001
From: Brandon Wu
Date: Sun, 6 Jul 2025 00:42:02 -0700
Subject: [PATCH] [clang][RISCV] Fix crash on VLS calling convention
This patch handl
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-8?q?Heidekr=C3=BCger?= ,
Paul =?utf-
=��___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heide
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heide
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heidekrüger?= ,
Paul =?utf-8?q?Heide
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff HEAD~1 HEAD --extensions cpp,c --
clang/lib/CodeGen/CGCall.cpp
clang/test/CodeGen/RI
https://github.com/4vtomat edited
https://github.com/llvm/llvm-project/pull/145489
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/4vtomat updated
https://github.com/llvm/llvm-project/pull/145489
>From 8b90033dd07ec2256f26670a43a0f713cc7b84ef Mon Sep 17 00:00:00 2001
From: Brandon Wu
Date: Sun, 6 Jul 2025 00:42:02 -0700
Subject: [PATCH] [clang][RISCV] Fix crash on VLS calling convention
This patch handl
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/146970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,52 @@
+//==-- AvoidPlatformSpecificFundamentalTypesCheck.h - clang-tidy -*- 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: Apa
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/146830
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,111 @@
+//===--- UseConcisePreprocessorDirectivesCheck.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: Ap
@@ -0,0 +1,40 @@
+//===--- UseConcisePreprocessorDirectivesCheck.h - clang-tidy ---*- 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: Apa
kikairoya wrote:
> > a symlink alone won't suffice here, as `dlopen()` ultimately calls
> > `LoadLibraryExW`, which does not follow Cygwin-style symlinks.
>
> If dlopen doesn't work on a Cygwin symlink, that is a bug in dlopen and
> should be fixed in Cygwin.
I had been misunderstood the func
https://github.com/kikairoya edited
https://github.com/llvm/llvm-project/pull/147132
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kikairoya updated
https://github.com/llvm/llvm-project/pull/147132
>From 9198e98e6dd7a02c5c5b65e400d788b6bac3a3c3 Mon Sep 17 00:00:00 2001
From: kikairoya
Date: Sat, 28 Jun 2025 13:11:42 +0900
Subject: [PATCH 1/3] [libclang][Cygwin] Put unversioned DLL file aside of
versione
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`sanitizer-x86_64-linux-fast` running on `sanitizer-buildbot3` while building
`clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/169/builds/12888
Here is the relevant piece
vbvictor wrote:
I like `IgnoreAliasing`. That is what user most likely understand with ease.
`SugaredTypes` is more clang developer-ish indeed.
https://github.com/llvm/llvm-project/pull/147060
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/147060
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,52 @@
+//==-- AvoidPlatformSpecificFundamentalTypesCheck.h - clang-tidy -*- 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: Apa
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/146970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,147 @@
+// RUN: %check_clang_tidy %s
portability-avoid-platform-specific-fundamental-types %t --
-config="{CheckOptions: [{key:
portability-avoid-platform-specific-fundamental-types.WarnOnChars, value:
false}, {key:
portability-avoid-platform-specific-fundamental-t
@@ -0,0 +1,121 @@
+//===--- AvoidPlatformSpecificFundamentalTypesCheck.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-Ide
@@ -0,0 +1,260 @@
+//===--- AvoidPlatformSpecificFundamentalTypesCheck.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: Ap
https://github.com/vbvictor commented:
Please remove unnecessary comments that tell the same as code itself, speaking
mostly about `registerMatchers`.
https://github.com/llvm/llvm-project/pull/146970
___
cfe-commits mailing list
cfe-commits@lists.llvm
@@ -0,0 +1,52 @@
+//==-- AvoidPlatformSpecificFundamentalTypesCheck.h - clang-tidy -*- 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: Apa
@@ -0,0 +1,260 @@
+//===--- AvoidPlatformSpecificFundamentalTypesCheck.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: Ap
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/146970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,104 @@
+// RUN: %check_clang_tidy %s
portability-avoid-platform-specific-fundamental-types %t --
-config="{CheckOptions: [{key:
portability-avoid-platform-specific-fundamental-types.WarnOnInts, value:
false}, {key:
portability-avoid-platform-specific-fundamental-ty
@@ -0,0 +1,260 @@
+//===--- AvoidPlatformSpecificFundamentalTypesCheck.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: Ap
@@ -0,0 +1,43 @@
+//===--- AvoidPlatformSpecificFundamentalTypesCheck.h - clang-tidy ---*-
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-Ide
@@ -0,0 +1,69 @@
+// RUN: %check_clang_tidy %s
portability-avoid-platform-specific-fundamental-types %t --
-config="{CheckOptions: [{key:
portability-avoid-platform-specific-fundamental-types.WarnOnInts, value:
false}, {key:
portability-avoid-platform-specific-fundamental-typ
@@ -0,0 +1,43 @@
+//===--- AvoidPlatformSpecificFundamentalTypesCheck.h - clang-tidy ---*-
C++
+//-*-===//
vbvictor wrote:
Please format this as one-line
https://github.com/llvm/llvm-project/pull/146970
___
cfe-co
https://github.com/5chmidti approved this pull request.
LGTM minus the open conversation
https://github.com/llvm/llvm-project/pull/146830
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
5chmidti wrote:
The option name could also be something like `IgnoreAliasing` or something
similar. You're also missing changes to the check docs itself.
https://github.com/llvm/llvm-project/pull/147060
___
cfe-commits mailing list
cfe-commits@lists.l
kikairoya wrote:
> LGTM, but can you amend the PR description with the extra info you gathered,
> about why this hasn't been an issue in other existing configurations?
Thanks.
I've update the description.
> > It seems `AND NOT CYGWIN` should be added here.
>
> That sounds like a good additi
https://github.com/owenca closed
https://github.com/llvm/llvm-project/pull/147164
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Owen Pan
Date: 2025-07-06T14:19:30-07:00
New Revision: 5ccbea9f480d5c387c7f21aa101e72774c91f1d3
URL:
https://github.com/llvm/llvm-project/commit/5ccbea9f480d5c387c7f21aa101e72774c91f1d3
DIFF:
https://github.com/llvm/llvm-project/commit/5ccbea9f480d5c387c7f21aa101e72774c91f1d3.diff
LOG:
https://github.com/kikairoya edited
https://github.com/llvm/llvm-project/pull/147108
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 232 matches
Mail list logo
