https://github.com/labrinea approved this pull request.
We can debate on what the sensible behavior ought to be, but I agree having a
test in place to show the current behavior is a good idea. Thanks for writing
one.
https://github.com/llvm/llvm-project/pull/143570
Author: David Green
Date: 2025-06-11T09:50:26+01:00
New Revision: ddb771ecfd12cab8d323a4e64e64b965883585de
URL:
https://github.com/llvm/llvm-project/commit/ddb771ecfd12cab8d323a4e64e64b965883585de
DIFF:
https://github.com/llvm/llvm-project/commit/ddb771ecfd12cab8d323a4e64e64b965883585de.diff
L
https://github.com/davemgreen closed
https://github.com/llvm/llvm-project/pull/142760
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ostannard approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/142760
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-amdgpu-runtime-2` running on `rocm-worker-hw-02` while building
`clang,openmp` at step 6 "test-openmp".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/10/builds/7099
Here is the relev
@@ -0,0 +1,23 @@
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -DNO_CROSSCHECK -verify
%s
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -analyzer-config
crosscheck-with-z3=true -verify %s
+// REQUIRES: z3
+
+// The SMTConv layer did not comprehend _BitInt types
eZWALT wrote:
> I originally kept the `NumGeneratedLoops` information consistent despite
> being partially subsumed by NumGeneratedLoopNests (Note that its not actually
> the same, it returns the number of generated loops in total adding nested
> loops, but due to the current usage of this sem
Vipul-Cariappa wrote:
Hi @SahilPatidar, I see that this PR was reverted. Any ideas on when it will be
re-applied?
https://github.com/llvm/llvm-project/pull/110418
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
MaggieYingYi wrote:
I had created https://github.com/llvm/llvm-project/pull/143614 for code review
in order to reload the patch.
https://github.com/llvm/llvm-project/pull/142409
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llv
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Corentin Jabot (cor3ntin)
Changes
Per [decl.ref],
> Because a null pointer value or a pointer past the end of an object
does not point to an object, a reference in a well-defined program cannot refer
to such things.
Note this does not fi
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/143667
Per [decl.ref],
> Because a null pointer value or a pointer past the end of an object
does not point to an object, a reference in a well-defined program cannot refer
to such things.
Note this does not fixes t
https://github.com/tblah closed https://github.com/llvm/llvm-project/pull/143508
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -195,6 +195,52 @@ declaration kind), all of which are optional:
SwiftReleaseOp: immortal
SwiftRetainOp: immortal
+:SwiftReturnOwnership:
+
+ Specifies the ownership convention of a function or method returning a C++
type
+ that has been imported as a Swift re
cor3ntin wrote:
@tbaederr
https://github.com/llvm/llvm-project/pull/143667
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Juan Manuel Martinez =?utf-8?q?Caamaño?= ,
Juan Manuel Martinez =?utf-8?q?Caamaño?= ,
Juan Manuel Martinez =?utf-8?q?Caamaño?=
Message-ID:
In-Reply-To:
@@ -0,0 +1,98 @@
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -
@@ -195,6 +195,52 @@ declaration kind), all of which are optional:
SwiftReleaseOp: immortal
SwiftRetainOp: immortal
+:SwiftReturnOwnership:
+
+ Specifies the ownership convention of a function or method returning a C++
type
+ that has been imported as a Swift re
Author: Dmitry Polukhin
Date: 2025-06-11T10:35:06+01:00
New Revision: 9797b5fcfbb9b9c96a219985f3623849bbd3956e
URL:
https://github.com/llvm/llvm-project/commit/9797b5fcfbb9b9c96a219985f3623849bbd3956e
DIFF:
https://github.com/llvm/llvm-project/commit/9797b5fcfbb9b9c96a219985f3623849bbd3956e.dif
dmpolukhin wrote:
@vbvictor and @HerrCai0907 @carlosgalvezp @PiotrZSL please take another look.
https://github.com/llvm/llvm-project/pull/140912
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
https://github.com/tmatheson-arm approved this pull request.
Sorry for the slow responses, this has revealed some issues that we are not
sure how to fix... I am happy with the patch as it is, it is a strict
improvement for `reconstructFromParsedFeatures`.
As @labrinea pointed out though, you s
https://github.com/dmpolukhin closed
https://github.com/llvm/llvm-project/pull/143168
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vabridgers updated
https://github.com/llvm/llvm-project/pull/143310
>From 66fbfd292123136e87528d710db6aeeceed3cce2 Mon Sep 17 00:00:00 2001
From: Vince Bridgers
Date: Sun, 8 Jun 2025 15:48:04 +0200
Subject: [PATCH 1/4] [analyzer] Correct SMT Layer for _BitInt cases
refutatio
https://github.com/Backl1ght updated
https://github.com/llvm/llvm-project/pull/143244
>From 51177d373e726b001bafaf6c2f8242b24136f608 Mon Sep 17 00:00:00 2001
From: Backl1ght
Date: Sat, 7 Jun 2025 16:15:04 +0800
Subject: [PATCH 1/4] add visit check
---
clang/lib/Sema/SemaDeclCXX.cpp | 14
@@ -570,23 +572,37 @@ class SMTConv {
// TODO: Refactor to put elsewhere
static inline QualType getAPSIntType(ASTContext &Ctx,
const llvm::APSInt &Int) {
-return Ctx.getIntTypeForBitwidth(Int.getBitWidth(), Int.isSigned());
+Qu
@@ -570,23 +572,37 @@ class SMTConv {
// TODO: Refactor to put elsewhere
static inline QualType getAPSIntType(ASTContext &Ctx,
const llvm::APSInt &Int) {
-return Ctx.getIntTypeForBitwidth(Int.getBitWidth(), Int.isSigned());
+Qu
@@ -0,0 +1,23 @@
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -DNO_CROSSCHECK -verify
%s
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -analyzer-config
crosscheck-with-z3=true -verify %s
+// REQUIRES: z3
+
+// The SMTConv layer did not comprehend _BitInt types
@@ -0,0 +1,23 @@
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -DNO_CROSSCHECK -verify
%s
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -analyzer-config
crosscheck-with-z3=true -verify %s
+// REQUIRES: z3
+
+// The SMTConv layer did not comprehend _BitInt types
@@ -0,0 +1,23 @@
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -DNO_CROSSCHECK -verify
%s
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -analyzer-config
crosscheck-with-z3=true -verify %s
+// REQUIRES: z3
+
+// The SMTConv layer did not comprehend _BitInt types
vabridgers wrote:
Thanks @NagyDonat and @steakhal for the useful comments. I think all have been
resolved at this point. If I've missed something I'll address. Best.
https://github.com/llvm/llvm-project/pull/143310
___
cfe-commits mailing list
cfe-com
@@ -0,0 +1,19 @@
+// RUN: rm -rf %t
+// RUN: mkdir -p %t
+
+// Create PCH without -ignore-pch.
+// RUN: %clang -x c++-header %S/../Modules/Inputs/codegen-flags/foo.h -###
2>&1 | FileCheck %s -check-prefix=CHECK-EMIT-PCH
MaggieYingYi wrote:
Thanks, fixed in
http
@@ -0,0 +1,98 @@
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=CL -finclude-default-header %s
+// RUN: %clang_cc1 %s -triple amdgcn-unknown-unknown -verify -pedantic
-Wconversion -Werror -fsyntax-only -cl-std=C
@@ -1547,6 +1547,38 @@ struct FormatStyle {
bool BeforeWhile;
/// Indent the wrapped braces themselves.
bool IndentBraces;
+/// Indent nested wrapped lambda braces.
rmarker wrote:
I had an idea about the name.
When looking again at the projects
@@ -1547,6 +1547,38 @@ struct FormatStyle {
bool BeforeWhile;
/// Indent the wrapped braces themselves.
bool IndentBraces;
+/// Indent nested wrapped lambda braces.
rmarker wrote:
Created a patch to upgrade `IndentBraces`. #143663
https://git
https://github.com/shafik updated
https://github.com/llvm/llvm-project/pull/143578
>From 09060e9a91842346f106d55b32141b16efe7378d Mon Sep 17 00:00:00 2001
From: Shafik Yaghmour
Date: Tue, 10 Jun 2025 10:41:04 -0700
Subject: [PATCH 1/3] [Clang][ByteCode][NFC] Move APInt into pushInteger since
i
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/143661
When the definition of a special member function was defaulted we would not
consider it user-provided, even when the first declaration was not defaulted.
Fixes #143599
>From b12415e6d4bfab4d97a8a416c5f924e0bb
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/143661
>From b12415e6d4bfab4d97a8a416c5f924e0bb8d62ce Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Wed, 11 Jun 2025 09:21:05 +0200
Subject: [PATCH 1/2] [Clang] _default-movable_ should be based on the first
de
@@ -195,6 +195,52 @@ declaration kind), all of which are optional:
SwiftReleaseOp: immortal
SwiftRetainOp: immortal
+:SwiftReturnOwnership:
+
+ Specifies the ownership convention of a function or method returning a C++
type
+ that has been imported as a Swift re
Author: Tom Eccles
Date: 2025-06-11T10:21:07+01:00
New Revision: 7ffdf4240d62724dca7f42b37bd8671fefe17e17
URL:
https://github.com/llvm/llvm-project/commit/7ffdf4240d62724dca7f42b37bd8671fefe17e17
DIFF:
https://github.com/llvm/llvm-project/commit/7ffdf4240d62724dca7f42b37bd8671fefe17e17.diff
LO
@@ -0,0 +1,728 @@
+#include "clang/Analysis/Analyses/LifetimeSafety.h"
+#include "clang/AST/Decl.h"
+#include "clang/AST/Expr.h"
+#include "clang/AST/StmtVisitor.h"
+#include "clang/AST/Type.h"
+#include "clang/Analysis/AnalysisDeclContext.h"
+#include "clang/Analysis/CFG.h"
+#inc
@@ -2045,6 +2047,8 @@ static void DiagnoseNonDefaultMovable(Sema &SemaRef,
SourceLocation Loc,
<< Decl->isMoveAssignmentOperator() << Decl->getSourceRange();
}
CXXDestructorDecl *Dtr = D->getDestructor();
+ if (Dtr)
AaronBallman wrote:
I think
https://github.com/vogelsgesang edited
https://github.com/llvm/llvm-project/pull/141889
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vogelsgesang closed
https://github.com/llvm/llvm-project/pull/141889
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NagyDonat commented:
Thanks for the updates! I added a few more style nitpicks, but overall LGTM.
https://github.com/llvm/llvm-project/pull/143310
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
@@ -570,23 +572,35 @@ class SMTConv {
// TODO: Refactor to put elsewhere
static inline QualType getAPSIntType(ASTContext &Ctx,
const llvm::APSInt &Int) {
-return Ctx.getIntTypeForBitwidth(Int.getBitWidth(), Int.isSigned());
+Qu
@@ -570,23 +572,35 @@ class SMTConv {
// TODO: Refactor to put elsewhere
static inline QualType getAPSIntType(ASTContext &Ctx,
const llvm::APSInt &Int) {
-return Ctx.getIntTypeForBitwidth(Int.getBitWidth(), Int.isSigned());
+Qu
https://github.com/AaronBallman approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/143611
___
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/143603
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rorth updated
https://github.com/llvm/llvm-project/pull/142948
>From e57e53c7e5abdb4c390a04b4ce9084dec9e71dd5 Mon Sep 17 00:00:00 2001
From: Rainer Orth
Date: Thu, 5 Jun 2025 13:40:26 +0200
Subject: [PATCH 1/2] [clang][python][test] Move python binding tests to lit
framework
@@ -0,0 +1,22 @@
+def is_libclang_loadable():
+try:
+sys.path.append(os.path.join(config.clang_src_dir, "bindings/python"))
+from clang.cindex import Config
+conf = Config()
+Config.set_library_path(config.clang_lib_dir)
+conf.lib
+
@@ -1888,10 +1888,11 @@ namespace PR15884 {
}
namespace AfterError {
- constexpr int error() {
+ constexpr int error() { // pre-cxx23-error {{no return statement in
constexpr function}}
AaronBallman wrote:
Hmm okay, I didn't write it in the issue but I was
Author: Adrian Vogelsgesang
Date: 2025-06-11T13:50:32+02:00
New Revision: 756e7cfd86c7f2bf20aaa1a3f87b5aa72ec128b4
URL:
https://github.com/llvm/llvm-project/commit/756e7cfd86c7f2bf20aaa1a3f87b5aa72ec128b4
DIFF:
https://github.com/llvm/llvm-project/commit/756e7cfd86c7f2bf20aaa1a3f87b5aa72ec128b4
https://github.com/AaronBallman approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/143578
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rorth wrote:
Two issues are worth mentioning about the updated PR:
- Although I'd originally disabled the Clang Python tests on SPARC in [[python,
tests] Disable Clang Python tests on SPARC](https://reviews.llvm.org/D60046),
they now `PASS` on Solaris/sparcv9, while on Linux/sparc64 `python` `S
@@ -294,6 +294,34 @@ Value *CodeGenFunction::EmitPPCBuiltinExpr(unsigned
BuiltinID,
return Builder.CreateCall(F, Ops, "");
}
+ // BCD convert builtins for P9
+ case PPC::BI__builtin_ppc_national2packed:
+ case PPC::BI__builtin_ppc_packed2zoned:
+ case PPC::BI__built
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/143661
>From b12415e6d4bfab4d97a8a416c5f924e0bb8d62ce Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Wed, 11 Jun 2025 09:21:05 +0200
Subject: [PATCH 1/3] [Clang] _default-movable_ should be based on the first
de
@@ -0,0 +1,23 @@
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -DNO_CROSSCHECK -verify
%s
+// RUN: %clang_cc1 -analyze -analyzer-checker=core -w -analyzer-config
crosscheck-with-z3=true -verify %s
vabridgers wrote:
Updated, resolving.
https://github.c
https://github.com/SLTozer closed
https://github.com/llvm/llvm-project/pull/108214
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/143310
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,22 @@
+// RUN: %clang_analyze_cc1 -analyzer-checker=core,debug.ExprInspection -w \
+// RUN: -analyzer-config crosscheck-with-z3=true -verify %s
+// REQUIRES: z3
+
+// Previously these tests were crashing because the SMTConv layer did not
+// comprehend the _BitInt typ
@@ -570,23 +572,35 @@ class SMTConv {
// TODO: Refactor to put elsewhere
static inline QualType getAPSIntType(ASTContext &Ctx,
const llvm::APSInt &Int) {
-return Ctx.getIntTypeForBitwidth(Int.getBitWidth(), Int.isSigned());
+Qu
SLTozer wrote:
Closed in favour of a new patch stack for the same feature starting at:
https://github.com/llvm/llvm-project/pull/143590
https://github.com/llvm/llvm-project/pull/108214
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
@@ -570,23 +572,35 @@ class SMTConv {
// TODO: Refactor to put elsewhere
static inline QualType getAPSIntType(ASTContext &Ctx,
const llvm::APSInt &Int) {
-return Ctx.getIntTypeForBitwidth(Int.getBitWidth(), Int.isSigned());
+Qu
https://github.com/AaronBallman approved this pull request.
LGTM aside from a suggestion
https://github.com/llvm/llvm-project/pull/143661
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
rorth wrote:
> > am at a bit of a loss how to handle build_target now [...] Don't know
> > if/how I can test this myself.
>
> All the `check-clang-python` did is depend on the `libclang` target and then
> call unittest, so I think it should be enough to change the `build_target` to
> `libclan
https://github.com/vogelsgesang edited
https://github.com/llvm/llvm-project/pull/141889
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shafik closed
https://github.com/llvm/llvm-project/pull/143646
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/143162
___
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 c,cpp,h --
clang/test/CodeGen/RISCV/include.c clang/lib/Code
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/143192
>From 0a63b900e1b934e15273763d8947c3cea712fef6 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Fri, 6 Jun 2025 20:56:49 +0200
Subject: [PATCH 1/6] [CIR] Upstream CreateOp for ComplexType with folder
---
@@ -13518,8 +13518,29 @@ bool
Sema::GloballyUniqueObjectMightBeAccidentallyDuplicated(
// If the object isn't hidden, the dynamic linker will prevent duplication.
clang::LinkageInfo Lnk = Target->getLinkageAndVisibility();
- if (Lnk.getVisibility() != HiddenVisibility)
+
@@ -13518,8 +13518,29 @@ bool
Sema::GloballyUniqueObjectMightBeAccidentallyDuplicated(
// If the object isn't hidden, the dynamic linker will prevent duplication.
clang::LinkageInfo Lnk = Target->getLinkageAndVisibility();
- if (Lnk.getVisibility() != HiddenVisibility)
+
https://github.com/anchuraj updated
https://github.com/llvm/llvm-project/pull/143441
>From 51c6a6a9b1f58f37df3eb1b6988b62f489c8f4b8 Mon Sep 17 00:00:00 2001
From: Anchu Rajendran
Date: Wed, 4 Jun 2025 15:12:49 -0500
Subject: [PATCH 1/2] [flang][flang-driver] atomic control support
---
clang/i
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/143355
>From 566a4cf83ec98e3ab89727e15498002df6e093c0 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Sun, 8 Jun 2025 22:14:20 +0200
Subject: [PATCH 1/4] [CIR] Implement folder for VecCreateOp
---
clang/includ
Author: Kazu Hirata
Date: 2025-06-11T12:50:11-07:00
New Revision: e266d6a5da6871c89747416c70a4a39181b594fb
URL:
https://github.com/llvm/llvm-project/commit/e266d6a5da6871c89747416c70a4a39181b594fb
DIFF:
https://github.com/llvm/llvm-project/commit/e266d6a5da6871c89747416c70a4a39181b594fb.diff
L
@@ -13518,8 +13518,29 @@ bool
Sema::GloballyUniqueObjectMightBeAccidentallyDuplicated(
// If the object isn't hidden, the dynamic linker will prevent duplication.
clang::LinkageInfo Lnk = Target->getLinkageAndVisibility();
- if (Lnk.getVisibility() != HiddenVisibility)
+
https://github.com/anchuraj created
https://github.com/llvm/llvm-project/pull/143769
None
>From 51c6a6a9b1f58f37df3eb1b6988b62f489c8f4b8 Mon Sep 17 00:00:00 2001
From: Anchu Rajendran
Date: Wed, 4 Jun 2025 15:12:49 -0500
Subject: [PATCH 1/2] [flang][flang-driver] atomic control support
---
c
@@ -13518,8 +13518,29 @@ bool
Sema::GloballyUniqueObjectMightBeAccidentallyDuplicated(
// If the object isn't hidden, the dynamic linker will prevent duplication.
clang::LinkageInfo Lnk = Target->getLinkageAndVisibility();
- if (Lnk.getVisibility() != HiddenVisibility)
+
@@ -0,0 +1,39 @@
+//===--- MakeFunctionToDirectCheck.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: Apach
@@ -1873,6 +1873,17 @@ Parser::TryAnnotateName(CorrectionCandidateCallback *CCC,
return AnnotatedNameKind::Unresolved;
}
+SourceLocation Parser::getEndOfPreviousToken() const {
+ SourceLocation TokenEndLoc = PP.getLocForEndOfToken(PrevTokLocation);
+ if (TokenEndLoc.isVali
https://github.com/vbvictor requested changes to this pull request.
Generally good check idea, but needs some polishing.
- Neither `modernize-make-function-to-direct` nor `modernize-make-direct`
sounds clear to me (specifically `direct` part), the main purpose of this
transformation is to use `
https://github.com/ConcreteCactus updated
https://github.com/llvm/llvm-project/pull/130421
From 0e9da921d39573f5a7c920b2bdd572b83f194e7f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=81ron=20H=C3=A1rn=C3=A1si?=
Date: Fri, 22 Nov 2024 21:43:04 +0100
Subject: [PATCH] [clang-tidy] Added bugprone-un
@@ -124,6 +124,12 @@ New checks
pointer and store it as class members without handle the copy and move
constructors and the assignments.
+- New :doc:`modernize-make-direct `
check.
+
+ Converts std::make_* function calls to direct constructor calls using CTAD.
--
@@ -0,0 +1,46 @@
+.. title:: clang-tidy - modernize-make-direct
+
+modernize-make-direct
vbvictor wrote:
Sync check name accross all files, somewhere you have "MakeFunctionToDirect".
https://github.com/llvm/llvm-project/pull/118120
__
https://github.com/erichkeane created
https://github.com/llvm/llvm-project/pull/143751
PR #143720 adds a requirement to the ACC dialect that every acc.loop must have
a seq, independent, or auto attribute for the 'default' device_type. The
standard has rules for how this can be intuited:
orpha
@@ -893,13 +893,8 @@ void
MicrosoftCXXABI::emitVirtualObjectDelete(CodeGenFunction &CGF,
QualType ElementType,
const CXXDestructorDecl *Dtor) {
// FIXME: Provide a source location her
@@ -110,56 +145,113 @@ static std::string computeBaseSysRoot(const Driver &D,
bool IncludeTriple) {
return std::string(SysRootDir);
}
-BareMetal::BareMetal(const Driver &D, const llvm::Triple &Triple,
- const ArgList &Args)
-: ToolChain(D, Triple, Ar
Author: Jorge Gorbe Moya
Date: 2025-06-11T11:48:23-07:00
New Revision: 8dc63ca59003a4b72217221c1c801237614c9d7d
URL:
https://github.com/llvm/llvm-project/commit/8dc63ca59003a4b72217221c1c801237614c9d7d
DIFF:
https://github.com/llvm/llvm-project/commit/8dc63ca59003a4b72217221c1c801237614c9d7d.di
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/143460
>From dd4953312066cb63ae1a3882270426c87b1f5b7a Mon Sep 17 00:00:00 2001
From: Oleksandr Tarasiuk
Date: Tue, 10 Jun 2025 02:47:51 +0300
Subject: [PATCH 1/5] [Clang] fix missing source location for ':' error in
@@ -320,6 +321,69 @@ bool
clang::analyze_format_string::ParseUTF8InvalidSpecifier(
// Methods on ArgType.
//===--===//
+static bool namedTypeToLengthModifierKind(QualType QT,
+
https://github.com/sarnex closed
https://github.com/llvm/llvm-project/pull/142511
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/143751
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -21,6 +21,88 @@
using namespace clang;
using namespace clang::CIRGen;
+/// Checks whether the given constructor is a valid subject for the
+/// complete-to-base constructor delgation optimization, i.e. emitting the
+/// complete constructor as a simple call to the base cons
https://github.com/vgvassilev updated
https://github.com/llvm/llvm-project/pull/94166
>From 306ef834c83e358f3463a73dddacd1346864699e Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Thu, 30 May 2024 05:05:41 +
Subject: [PATCH 1/2] [clang-repl] Teach clang-repl how to load PCHs.
---
cl
https://github.com/DKLoehr updated
https://github.com/llvm/llvm-project/pull/143537
>From 8ff6b6828604b25ded852adba37dd82e5cf7f771 Mon Sep 17 00:00:00 2001
From: Devon Loehr
Date: Tue, 10 Jun 2025 13:57:06 +
Subject: [PATCH 1/3] Enable UOD warning for windows
---
clang/include/clang/Basic
https://github.com/smithp35 requested changes to this pull request.
Apologies, after further testing I think I've found a problem that can break
existing toolchains.
https://github.com/llvm/llvm-project/pull/121829
___
cfe-commits mailing list
cfe-com
https://github.com/anchuraj edited
https://github.com/llvm/llvm-project/pull/143441
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper updated
https://github.com/llvm/llvm-project/pull/143322
>From 78c98a97d272aa487337c3b9280388e3aeb5b354 Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Sun, 8 Jun 2025 21:08:14 +0200
Subject: [PATCH 1/4] [CIR] Implement folder for VecCmpOp
---
clang/include/c
https://github.com/arichardson closed
https://github.com/llvm/llvm-project/pull/139608
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -327,6 +330,27 @@ a discriminator determined as follows:
is ``ptrauth_blend_discriminator(&x, discriminator)``; see
`ptrauth_blend_discriminator`_.
+``ptrauth_qualifier_options``
+~
+
+The options parameter to the ``__ptrauth`` qualifier is a
@@ -0,0 +1,78 @@
+//===--- MakeFunctionToDirectCheck.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: Apach
@@ -0,0 +1,39 @@
+//===--- MakeFunctionToDirectCheck.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: Apach
1 - 100 of 440 matches
Mail list logo
