https://github.com/tbaederr approved this pull request.
https://github.com/llvm/llvm-project/pull/143034
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vbvictor requested changes to this pull request.
Since you de-facto added this option to `performance-noexcept-destructor` check
too, please add it to `performance-noexcept-destructor.rst` docs (just
copypaste), add to ReleaseNotes and add a separate test-file for noexcept
d
@@ -0,0 +1,46 @@
+// RUN: %check_clang_tidy -check-suffix=CONFIG %s
performance-noexcept-move-constructor,performance-noexcept-destructor %t -- \
+// RUN: -config="{CheckOptions:
{performance-noexcept-move-constructor.AllowFalseEvaluated: true}}" \
+// RUN: -- -fexceptions
+
+na
@@ -33,6 +34,10 @@ class NoexceptFunctionBaseCheck : public ClangTidyCheck {
return TK_IgnoreUnlessSpelledInSource;
}
+ void storeOptions(ClangTidyOptions::OptionMap &Opts) override {
vbvictor wrote:
Please place definition in .cpp file
https://github
@@ -22,7 +22,8 @@ namespace clang::tidy::performance {
class NoexceptFunctionBaseCheck : public ClangTidyCheck {
public:
NoexceptFunctionBaseCheck(StringRef Name, ClangTidyContext *Context)
- : ClangTidyCheck(Name, Context) {}
+ : ClangTidyCheck(Name, Context),
@@ -110,6 +110,12 @@ Changes in existing checks
` check by providing additional
examples and fixing some macro related false positives.
+- Improved :doc:`performance-noexcept-move-constructor
+ ` check by adding
+ a new (off-by-default) option `AllowFalseEvaluated`, whic
@@ -253,6 +253,12 @@ Changes in existing checks
` check by fixing
some false positives involving smart pointers to arrays.
+- Improved :doc:`performance-noexcept-move-constructor
vbvictor wrote:
Please place this in alphabetical order (by check name).
Sho
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/126897
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/138293
>From 633b22215dd7b4addf5a9e1718a76030dd37f9e9 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Fri, 2 May 2025 17:24:13 +0200
Subject: [PATCH] [libc++] Upgrade to GCC 15
---
.github/workflows/libcxx-bu
https://github.com/zyn0217 ready_for_review
https://github.com/llvm/llvm-project/pull/143096
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,46 @@
+// RUN: %check_clang_tidy -check-suffix=CONFIG %s
performance-noexcept-move-constructor,performance-noexcept-destructor %t -- \
+// RUN: -config="{CheckOptions:
{performance-noexcept-move-constructor.AllowFalseEvaluated: true}}" \
+// RUN: -- -fexceptions
+
+na
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Younan Zhang (zyn0217)
Changes
CWG2369 revealed another case where we were SFINAE'ing out the invalid result
of substitution, but the expression now makes into evaluation.
We switch to the concept specialization's context before we check
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/143096
>From 020160fde7a3cc13b104a5f5577a8037cf412c66 Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Fri, 6 Jun 2025 17:09:08 +0800
Subject: [PATCH 1/2] [Clang] Fail the constraint substitution early after
CWG2369
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/143096
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/143096
>From 020160fde7a3cc13b104a5f5577a8037cf412c66 Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Fri, 6 Jun 2025 17:09:08 +0800
Subject: [PATCH 1/2] [Clang] Fail the constraint substitution early after
CWG2369
https://github.com/kikairoya created
https://github.com/llvm/llvm-project/pull/143115
Handling of va_list on Cygwin environment must be matched to normal Windows
environment.
A new test file is added as existing test contains a unsupported functionality.
>From c64472a188825a1d814ea0523a6097d25
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
@llvm/pr-subscribers-clang
Author: Tomohiro Kashiwada (kikairoya)
Changes
Handling of va_list on Cygwin environment must be matched to normal Windows
environment.
A new test file is added as existing test contains a unsupported functionality
https://github.com/kikairoya created
https://github.com/llvm/llvm-project/pull/143117
On Cygwin environment, wint_t is unsigned int as shown here:
```
$ echo | gcc -m32 -xc - -E -dM | grep WINT_T
145:#define __SIZEOF_WINT_T__ 4
315:#define __WINT_TYPE__ unsigned int
```
```
$ echo | gcc -xc - -
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Tomohiro Kashiwada (kikairoya)
Changes
On Cygwin environment, wint_t is unsigned int as shown here:
```
$ echo | gcc -m32 -xc - -E -dM | grep WINT_T
145:#define __SIZEOF_WINT_T__ 4
315:#define __WINT_TYPE__ unsigned int
```
```
$ echo | gc
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/141305
>From 77641f88c5047c990df649658b6bd5fba0d395cd Mon Sep 17 00:00:00 2001
From: Oleksandr Tarasiuk
Date: Sat, 31 May 2025 15:59:59 +0300
Subject: [PATCH 1/2] [Clang] add fix-it hints for unknown attributes
---
Fznamznon wrote:
/cherry-pick 40417915a161e87b398f1cc3e9b7c159207abd77
https://github.com/llvm/llvm-project/pull/135957
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Fznamznon milestoned
https://github.com/llvm/llvm-project/pull/135957
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
LGTM!
https://github.com/llvm/llvm-project/pull/142983
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/egorshamshura updated
https://github.com/llvm/llvm-project/pull/142341
>From 2e423a75e5ee8b5ecc450d4457fc677fd0c08ad2 Mon Sep 17 00:00:00 2001
From: Shamshura Egor
Date: Mon, 2 Jun 2025 07:25:26 +
Subject: [PATCH 1/9] [Clang] Added explanation why a is trivial copyable
e
https://github.com/xlauko closed
https://github.com/llvm/llvm-project/pull/143024
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/143087
Fixes https://github.com/llvm/llvm-project/issues/142608
>From f44f250259cd09b20f6ab9095fc3718537d51563 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Fri, 6 Jun 2025 03:56:59 -0400
Subject: [PATCH] [
Author: Henrich Lauko
Date: 2025-06-06T09:58:13+02:00
New Revision: f10c7d9f5ace5dc55918ac3b86a2b06b0e358735
URL:
https://github.com/llvm/llvm-project/commit/f10c7d9f5ace5dc55918ac3b86a2b06b0e358735
DIFF:
https://github.com/llvm/llvm-project/commit/f10c7d9f5ace5dc55918ac3b86a2b06b0e358735.diff
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/136323
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,46 @@
+// RUN: %check_clang_tidy -check-suffix=CONFIG %s
performance-noexcept-move-constructor,performance-noexcept-destructor %t -- \
+// RUN: -config="{CheckOptions:
{performance-noexcept-move-constructor.AllowFalseEvaluated: true}}" \
+// RUN: -- -fexceptions
+
+na
https://github.com/vbvictor edited
https://github.com/llvm/llvm-project/pull/126897
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/138482
>From ecea44f2965cda3a4e6e1ccec096066830ddfa41 Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Sun, 4 May 2025 21:07:33 -0700
Subject: [PATCH 1/2] [clang][ptrauth] Add support for querying the ptrauth
schema of
Author: Naveen Seth Hanig
Date: 2025-06-06T09:47:34+02:00
New Revision: f482b9677eb8e6f1b12ec74225dfe0d3c78ec854
URL:
https://github.com/llvm/llvm-project/commit/f482b9677eb8e6f1b12ec74225dfe0d3c78ec854
DIFF:
https://github.com/llvm/llvm-project/commit/f482b9677eb8e6f1b12ec74225dfe0d3c78ec854.d
Nechda wrote:
I've rebased the branch, gentle pings to reviewers.
https://github.com/llvm/llvm-project/pull/126897
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/141695
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
xlauko wrote:
### Merge activity
* **Jun 6, 7:57 AM UTC**: A user started a stack merge that includes this pull
request via
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/143024).
https://github.com/llvm/llvm-project/pull/143024
__
https://github.com/dominik-steenken updated
https://github.com/llvm/llvm-project/pull/142346
>From 318f0536ce71780f808ef70a1817af515f9861bd Mon Sep 17 00:00:00 2001
From: Dominik Steenken
Date: Mon, 26 May 2025 14:53:41 +0200
Subject: [PATCH 1/2] Align global strings according to data layout
W
dominik-steenken wrote:
Ok. As far as i can tell,
```C++
M->getDataLayout().getPrefTypeAlign(getInt8Ty())
```
should be functionally equivalent to
```C++
M->getDataLayout().getPreferredAlign(GV)
```
with the exception that the latter potentially preserves a preexisting
alignment on the `GV`, wh
zokrezyl wrote:
IMHO: I guess a relevant test case is to take some huge relevant project, maybe
LLVM itself and parse all their files and walk through all the nodes of the ast
through all the members/methods recursively
https://github.com/llvm/llvm-project/pull/142634
_
https://github.com/DeinAlptraum closed
https://github.com/llvm/llvm-project/pull/142634
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Zokre Zyl
Date: 2025-06-06T11:14:57+02:00
New Revision: e7cd6b48c007fadd499ad746e6b3239091fcfd48
URL:
https://github.com/llvm/llvm-project/commit/e7cd6b48c007fadd499ad746e6b3239091fcfd48
DIFF:
https://github.com/llvm/llvm-project/commit/e7cd6b48c007fadd499ad746e6b3239091fcfd48.diff
LOG
github-actions[bot] wrote:
@zokrezyl Congratulations on having your first Pull Request (PR) merged into
the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a
problem with a build
Author: Younan Zhang
Date: 2025-06-06T19:13:59+08:00
New Revision: c3b870eb3f4fb15403b49a976aea860e6da9084e
URL:
https://github.com/llvm/llvm-project/commit/c3b870eb3f4fb15403b49a976aea860e6da9084e
DIFF:
https://github.com/llvm/llvm-project/commit/c3b870eb3f4fb15403b49a976aea860e6da9084e.diff
https://github.com/zyn0217 closed
https://github.com/llvm/llvm-project/pull/143096
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/143111
Use the definition of trivially relocatable types to short-circuit some checks
for trivial_abi.
Note that this is mostly a no-op as there is a lot of overlap between
trivial_abi and trivial relocatability (i
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Corentin Jabot (cor3ntin)
Changes
Use the definition of trivially relocatable types to short-circuit some checks
for trivial_abi.
Note that this is mostly a no-op as there is a lot of overlap between
trivial_abi and trivial relocatabili
denzor200 wrote:
> Personally I don't think this is a `performance` check. Boolean operators are
> for bools; bitwise operators are for ints. I would put this in `readability`
> or `misc` instead.
I've put this in the performance section because evaluating only one
operand(which will be in mo
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/141305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,35 @@
+#ifndef LLVM_CLANG_BASIC_ATTRIBUTESCOPEINFO_H
erichkeane wrote:
This needs a file header.
https://github.com/llvm/llvm-project/pull/141305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https
https://github.com/erichkeane approved this pull request.
I think this is a strict improvement, and I think looks correct to me.
https://github.com/llvm/llvm-project/pull/141305
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.l
https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/142998
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -272,6 +273,26 @@ QualType ObjCMessageExpr::getCallReturnType(ASTContext
&Ctx) const {
return Ctx.getReferenceQualifiedType(this);
}
+std::pair
erichkeane wrote:
I think we're decades past 'maybe we'll support other kinds some day' :D
If you're willi
@@ -272,6 +273,26 @@ QualType ObjCMessageExpr::getCallReturnType(ASTContext
&Ctx) const {
return Ctx.getReferenceQualifiedType(this);
}
+std::pair
+ObjCMessageExpr::getUnusedResultAttr(ASTContext &Ctx) const {
+ // If the callee is marked nodiscard, return that attribute
+
https://github.com/abhina-sree updated
https://github.com/llvm/llvm-project/pull/138895
>From 425aa66b8a57eb49ad0c678587a20e81c0ae5433 Mon Sep 17 00:00:00 2001
From: Abhina Sreeskantharajan
Date: Wed, 7 May 2025 11:26:59 -0400
Subject: [PATCH] This patch enables the fexec-charset option to cont
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/143111
>From d4e5db938e4af98dda2dd4b518273fe339c64bfa Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Fri, 6 Jun 2025 13:09:40 +0200
Subject: [PATCH 1/2] [Clang] Non-polymorphic trivially relocatable types can
ha
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/143034
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/abhina-sree updated
https://github.com/llvm/llvm-project/pull/143174
>From a796217c30f70f2ded3505c1624649b7226ef5ff Mon Sep 17 00:00:00 2001
From: Abhina Sreeskantharajan
Date: Fri, 6 Jun 2025 12:16:52 -0400
Subject: [PATCH] refactor AutoConvert.h to remove MVS guard
---
cl
@@ -99,6 +99,132 @@ enum class SamplerFeedbackType : uint32_t {
const unsigned MinWaveSize = 4;
const unsigned MaxWaveSize = 128;
+// Definition of the various d3d12.h enumerations and flags. The definitions of
+// all values here correspond to their description in the d3d12.h
https://github.com/bogner edited
https://github.com/llvm/llvm-project/pull/143041
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bogner approved this pull request.
This makes sense to me. As a follow up, we should look into whether
BinaryFormat/DXContainer.h can get some of these enum values from here rather
than duplicating their values.
https://github.com/llvm/llvm-project/pull/143041
__
https://github.com/andykaylor created
https://github.com/llvm/llvm-project/pull/143195
This change adds the support for accessing a member of a base class from a
derived class object.
>From 6fb28507d57ce04faa712b630e308ba029230fd5 Mon Sep 17 00:00:00 2001
From: Andy Kaylor
Date: Fri, 30 May 2
https://github.com/mcinally updated
https://github.com/llvm/llvm-project/pull/142800
>From 918b853de8c43dacebecb42cafa6d3b8fec15b47 Mon Sep 17 00:00:00 2001
From: Cameron McInally
Date: Tue, 3 Jun 2025 11:12:42 -0700
Subject: [PATCH 1/9] [Driver] Move CommonArgs to a location visible by the
Fr
@@ -99,6 +99,132 @@ enum class SamplerFeedbackType : uint32_t {
const unsigned MinWaveSize = 4;
const unsigned MaxWaveSize = 128;
+// Definition of the various d3d12.h enumerations and flags. The definitions of
+// all values here correspond to their description in the d3d12.h
https://github.com/rniwa updated
https://github.com/llvm/llvm-project/pull/142485
>From e0d5e1092893d6e93ead44763d5789cba84c34f9 Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Mon, 2 Jun 2025 14:52:38 -0600
Subject: [PATCH 1/2] [WebKit checkers] Treat passing of a member variable
which is c
https://github.com/abhina-sree updated
https://github.com/llvm/llvm-project/pull/143174
>From cedf7e22c9d1ee7af74aa9c8e3e77c4d86fe7767 Mon Sep 17 00:00:00 2001
From: Abhina Sreeskantharajan
Date: Fri, 6 Jun 2025 12:16:52 -0400
Subject: [PATCH] refactor AutoConvert.h to remove MVS guard
---
cl
https://github.com/andykaylor closed
https://github.com/llvm/llvm-project/pull/143176
___
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: Andy Kaylor (andykaylor)
Changes
This change adds the support for accessing a member of a base class from a
derived class object.
---
Patch is 22.82 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/p
@@ -0,0 +1,69 @@
+//===--===//
+//
+// 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: Apac
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/143195
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1677,6 +1678,34 @@ bool CIRGenModule::verifyModule() const {
return mlir::verify(theModule).succeeded();
}
+// TODO(cir): this can be shared with LLVM codegen.
+CharUnits CIRGenModule::computeNonVirtualBaseClassOffset(
+const CXXRecordDecl *derivedClass, CastExpr::pa
@@ -1677,6 +1678,34 @@ bool CIRGenModule::verifyModule() const {
return mlir::verify(theModule).succeeded();
}
+// TODO(cir): this can be shared with LLVM codegen.
+CharUnits CIRGenModule::computeNonVirtualBaseClassOffset(
+const CXXRecordDecl *derivedClass, CastExpr::pa
@@ -32,6 +32,22 @@ static void baz() {
} // namespace call_args_checked
+namespace call_args_member {
+
+void consume(CheckedObj&);
+
+struct WrapperObj {
+ CheckedObj checked;
+ CheckedObj& checkedRef;
+ void foo() {
+consume(checked);
rniwa wrote:
B
https://github.com/tarunprabhu requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/142800
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,430 @@
+#include "Generators.h"
+#include "clang/Basic/Specifiers.h"
+#include "llvm/Support/JSON.h"
+
+using namespace llvm;
+using namespace llvm::json;
+
+namespace clang {
+namespace doc {
+
+class JSONGenerator : public Generator {
+public:
+ static const char *Fo
https://github.com/ilovepi approved this pull request.
I think this is a good start, modulo a couple fixes on reserving space in
json::Arrays. I'm fine if we keep incrementally adding testing rather than
doing it all up front. When we're done I imagine most of the YAML tests can be
converted,
@@ -0,0 +1,430 @@
+#include "Generators.h"
+#include "clang/Basic/Specifiers.h"
+#include "llvm/Support/JSON.h"
+
+using namespace llvm;
+using namespace llvm::json;
+
+namespace clang {
+namespace doc {
+
+class JSONGenerator : public Generator {
+public:
+ static const char *Fo
https://github.com/NuriAmari updated
https://github.com/llvm/llvm-project/pull/129736
>From c4c106951803890d373e13a02c36d24c715a67a2 Mon Sep 17 00:00:00 2001
From: Nuri Amari
Date: Mon, 3 Mar 2025 16:10:54 -0800
Subject: [PATCH 1/3] Move Clang distributd thin-lto codegen tests into their
own d
https://github.com/ilovepi approved this pull request.
LGTM once the last few nits are addressed.
https://github.com/llvm/llvm-project/pull/142273
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/c
@@ -44,6 +43,12 @@ inline Target* uncheckedDowncast(Source* source)
return static_cast(source);
}
+template
+Target* [[clang::annotate_type("webkit.pointerconversion")]]
newCastFunction(Source*);
rniwa wrote:
I don't think so. I added a test case for tha
https://github.com/rniwa updated
https://github.com/llvm/llvm-project/pull/141277
>From b29b369a5b26869916101e45aa4580a5f7de3907 Mon Sep 17 00:00:00 2001
From: Ryosuke Niwa
Date: Fri, 23 May 2025 11:42:20 -0700
Subject: [PATCH 1/2] [WebKit checkers] Add an annotation for pointer
conversion.
T
https://github.com/Bigcheese approved this pull request.
https://github.com/llvm/llvm-project/pull/142452
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cyndyishida closed
https://github.com/llvm/llvm-project/pull/142452
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Cyndy Ishida
Date: 2025-06-06T13:45:16-07:00
New Revision: 897b0301d2e2ff28d3976fe95b64be5a85815908
URL:
https://github.com/llvm/llvm-project/commit/897b0301d2e2ff28d3976fe95b64be5a85815908
DIFF:
https://github.com/llvm/llvm-project/commit/897b0301d2e2ff28d3976fe95b64be5a85815908.diff
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/5] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
https://github.com/mcinally updated
https://github.com/llvm/llvm-project/pull/142800
>From 918b853de8c43dacebecb42cafa6d3b8fec15b47 Mon Sep 17 00:00:00 2001
From: Cameron McInally
Date: Tue, 3 Jun 2025 11:12:42 -0700
Subject: [PATCH 1/9] [Driver] Move CommonArgs to a location visible by the
Fr
https://github.com/dpaoliello requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/142970
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -16,7 +16,7 @@
; argc is the first debug location.
; ASM: .Ldebug_loc1:
; ASM-NEXT: .quad .Lfunc_begin0-.Lfunc_begin0
-; ASM-NEXT: .quad [[argc_range_end]]-.Lfunc_begin0
+; ASM-NEXT: .quad .Ltmp3-.Lfunc_begin0
dpaoliello wrote:
Do you understand what
@@ -316,8 +316,13 @@ static llvm::Error parseRecord(const Record &R, unsigned
ID,
static llvm::Error parseRecord(const Record &R, unsigned ID,
llvm::StringRef Blob, CommentInfo *I) {
switch (ID) {
- case COMMENT_KIND:
-return decodeRecord(
zokrezyl wrote:
> Great, thank you!
>
> > IMHO: I guess a relevant test case is to take some huge relevant project,
> > maybe LLVM itself and parse all their files and walk through all the nodes
> > of the ast through all the members/methods recursively
>
> The problem is that this depends on
@@ -670,12 +671,17 @@ static std::unique_ptr genHTML(const
CommentInfo &I) {
return nullptr;
return std::move(BlockComment);
}
- if (I.Kind == "TextComment") {
-if (I.Text == "")
+
+ case CommentKind::CK_TextComment: {
+if (I.Text.empty())
return n
@@ -0,0 +1,40 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --output=%t --format=json --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/MyClass.json
+
+struct MyClass {
+ template T methodTemplate(T param) {
+ }
+};
+
+// CHECK: "PublicFunct
https://github.com/evelez7 updated
https://github.com/llvm/llvm-project/pull/142483
>From fa8b80f9bfe2b7faf765ed4cf60fb8cec30e1d48 Mon Sep 17 00:00:00 2001
From: Erick Velez
Date: Mon, 2 Jun 2025 12:53:36 -0700
Subject: [PATCH 1/5] [clang-doc] add a JSON generator
---
clang-tools-extra/clang-
@@ -238,6 +238,12 @@
#define LLVM_PREFETCH(addr, rw, locality)
#endif
+#if __has_attribute(uninitialized)
+#define LLVM_ATTRIBUTE_UNINITIALIED __attribute__((uninitialized))
tarunprabhu wrote:
Spelling:
```suggestion
#define LLVM_ATTRIBUTE_UNINITIALIZED __a
@@ -0,0 +1,194 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --output=%t --format=json --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/MyClass.json
+
+struct Foo;
+
+// This is a nice class.
+// It has some nice methods and fields.
+// @brief This is
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 h,cpp --
clang-tools-extra/clang-doc/JSONGenerator.cpp
clan
@@ -0,0 +1,194 @@
+// RUN: rm -rf %t && mkdir -p %t
+// RUN: clang-doc --output=%t --format=json --executor=standalone %s
+// RUN: FileCheck %s < %t/GlobalNamespace/MyClass.json
+
+struct Foo;
+
+// This is a nice class.
+// It has some nice methods and fields.
+// @brief This is
@@ -27,3 +27,59 @@ float _Complex cf2 = { 1.0f, 2.0f };
// OGCG: {{.*}} = global { float, float } zeroinitializer, align 4
// OGCG: {{.*}} = global { i32, i32 } { i32 1, i32 2 }, align 4
// OGCG: {{.*}} = global { float, float } { float 1.00e+00, float
2.00e+00 }, alig
https://github.com/andykaylor commented:
This looks good. Just a request for a couple more test cases.
https://github.com/llvm/llvm-project/pull/143192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/malavikasamak created
https://github.com/llvm/llvm-project/pull/143205
None
>From 52e4413ea1e701dfe0b24cf957a26bb72732f066 Mon Sep 17 00:00:00 2001
From: MalavikaSamak
Date: Wed, 21 May 2025 16:06:44 -0700
Subject: [PATCH] Place holder message for sizeof operator in loops.
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Malavika Samak (malavikasamak)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/143205.diff
3 Files Affected:
- (modified) clang-tools-extra/clang-tidy/bugprone/SizeofExpressionCheck.cpp
(+16-1)
- (modifie
zygoloid wrote:
@PiotrZSL @5chmidti @HerrCai0907 Ping again. This has been waiting for review
for a few months, could someone take a look?
https://github.com/llvm/llvm-project/pull/130369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
1 - 100 of 545 matches
Mail list logo
