ranapratap55 wrote:
> If the idea here is to remove the need for
> https://github.com/ROCm/llvm-project/blob/amd-staging/amd/device-libs/ockl/src/base-image-intrinsics.ll
> and
> https://github.com/ROCm/llvm-project/blob/amd-staging/amd/device-libs/ockl/src/extended-image-intrinsics.ll
> then
@@ -1319,6 +1319,9 @@ def CmpOp : CIR_Op<"cmp", [Pure, SameTypeOperands]> {
`cir.bool` result. The kinds of comparison available are:
[lt,gt,ge,eq,ne]
+Note: The 'complex' dialect has separate complex.eq and complex.neq
+operations
+
xlauko wro
@@ -621,6 +621,32 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned
BuiltinID,
llvm::Function *F = CGM.getIntrinsic(IID, {LoadTy});
return Builder.CreateCall(F, {Addr});
}
+ case AMDGPU::BI__builtin_amdgcn_tensor_load_to_lds:
+ case AMDGPU::BI__builtin_amdg
@@ -0,0 +1,562 @@
+// RUN: %check_clang_tidy -std=c++11 -check-suffix=11 %s misc-use-constexpr %t
-- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy -std=c++14 -check-suffix=11,14 %s misc-use-constexpr
%t -- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy
https://github.com/dpaoliello approved this pull request.
https://github.com/llvm/llvm-project/pull/146456
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -227,26 +227,24 @@ class CIRBaseBuilderTy : public mlir::OpBuilder {
//======//
cir::CallOp createCallOp(mlir::Location loc, mlir::SymbolRefAttr callee,
- mlir::Type returnTyp
https://github.com/efriedma-quic commented:
>From an ABI perspective, this seems fine: if you want fp to always be valid,
>sure, lets enforce that.
Do we generate appropriate diagnostics for inline asm?
Does the AArch64 backend actually support this flag? I briefly tried, and it
looks like t
https://github.com/bcardosolopes edited
https://github.com/llvm/llvm-project/pull/145178
___
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/146577
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/changpeng closed
https://github.com/llvm/llvm-project/pull/146409
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bcardosolopes edited
https://github.com/llvm/llvm-project/pull/145178
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -463,6 +463,27 @@ class Triple {
const std::string &str() const { return Data; }
+ /// Return the triple string but only keep the first \p N components.
+ ///
+ /// The returned string will preserve the first \p N components exactly the
+ /// same as the original (in
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/145157
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Changpeng Fang
Date: 2025-07-01T11:08:49-07:00
New Revision: 5035d20dcbcea1edced779148ac69b84d3c97577
URL:
https://github.com/llvm/llvm-project/commit/5035d20dcbcea1edced779148ac69b84d3c97577
DIFF:
https://github.com/llvm/llvm-project/commit/5035d20dcbcea1edced779148ac69b84d3c97577.diff
@@ -662,13 +665,17 @@ static void printCallCommon(mlir::Operation *op,
}
printer << "(" << ops << ")";
+ if (isNothrow)
+printer << " nothrow";
bcardosolopes wrote:
and here!
https://github.com/llvm/llvm-project/pull/145178
_
@@ -611,6 +611,9 @@ static mlir::ParseResult parseCallCommon(mlir::OpAsmParser
&parser,
if (parser.parseRParen())
return mlir::failure();
+ if (parser.parseOptionalKeyword("nothrow").succeeded())
+result.addAttribute("nothrow", mlir::UnitAttr::get(parser.getContext
https://github.com/bcardosolopes commented:
Thanks for the changes, more comments!
https://github.com/llvm/llvm-project/pull/145178
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -77,17 +77,38 @@ void CIRGenFunction::emitAggregateStore(mlir::Value value,
Address dest) {
builder.createStore(*currSrcLoc, value, dest);
}
+static void addAttributesFromFunctionProtoType(CIRGenBuilderTy &builder,
+ mlir::Na
https://github.com/dpaoliello created
https://github.com/llvm/llvm-project/pull/146582
There is no way in Arm64 Windows to indicate that a given function has used the
Frame Pointer as a General Purpose Register, as such stack walks will always
assume that the frame chain is valid and will foll
@@ -7246,337 +7248,339 @@ class Parser : public CodeCompletionHandler {
ParseStatementOrDeclaration(StmtVector &Stmts, ParsedStmtContext StmtCtx,
SourceLocation *TrailingElseLoc = nullptr);
- StmtResult ParseStatementOrDeclarationAfterAttribute
@@ -35,7 +36,7 @@ void f(int n) {
[[fallthrough]];
}
case 8:
-[[fallthrough]]; // expected-error {{does not directly precede switch
label}}
+[[fallthrough]]; // FIXME: The error {{does not directly precede switch
label}} cannot reproduce in this translate
@@ -0,0 +1,18 @@
+// RUN: %clang_cc1 -fsyntax-only -std=c2x -verify %s
AaronBallman wrote:
```suggestion
// RUN: %clang_cc1 -fsyntax-only -std=c23 -verify %s
```
Maybe rename the file to be `c23` instead of `c2x` as well?
https://github.com/llvm/llvm-project/pul
lenary wrote:
This seems to only implement the first bullet point in your commit message, and
assembler/relocation/fixup support, right? Please update the PR description.
I presume there will be follow-ups for the codegen/load-store support, plus
subtarget info.
https://github.com/llvm/llvm
@@ -15,18 +15,19 @@ void f(int n) {
return;
}
case 2:
+// FIXME: Should we emit an error {{fallthrough annotation does not
directly precede switch label}}?
AaronBallman wrote:
Yeah that seems like a regression in behavior.
https://github.com/
anthonyhatran wrote:
> Looks like we need to fix a few more things. Did you resolve the issue with
> the existing test cases failing?
Apologies for the belated reply on this one; I ran the test suite overnight
just to double-check that the test cases I listed were actually caused by my
change
5chmidti wrote:
Thanks for testing this on your codebase so fast as well.
> I got some fixits that do not compile, e.g.:
>
> ```
> std::erase_if(numbers, [](int n) {
> return n % 2 == 0;
> });
> ```
>
> is changed into
>
> ```
> std::erase_if(numbers, [](int nconstexpr ) {
> return n % 2 == 0
Author: Richard Smith
Date: 2025-07-01T13:31:46-07:00
New Revision: c56c349d39464d859b0f8655a4502e5c06924b66
URL:
https://github.com/llvm/llvm-project/commit/c56c349d39464d859b0f8655a4502e5c06924b66
DIFF:
https://github.com/llvm/llvm-project/commit/c56c349d39464d859b0f8655a4502e5c06924b66.diff
https://github.com/vogelsgesang updated
https://github.com/llvm/llvm-project/pull/141937
>From 185be04e482344ce2f76eeb02859162e1838150e Mon Sep 17 00:00:00 2001
From: Adrian Vogelsgesang
Date: Tue, 27 May 2025 19:36:42 +
Subject: [PATCH 1/6] [debuginfo][coro] Emit debug info labels for coro
https://github.com/zygoloid closed
https://github.com/llvm/llvm-project/pull/130369
___
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
@llvm/pr-subscribers-llvm-support
Author: None (vabridgers)
Changes
PR145731 corrected the analyzer test runner to consider use of z3 when used by
testcases, which exposed problems in test cases PR37855.c and crashes in
z3-crosscheck.c This chang
@@ -3639,6 +3639,64 @@ make the function's CFI jump table canonical. See
:ref:`the CFI documentation
}];
}
+def CFISaltDocs : Documentation {
+ let Category = DocCatFunction;
+ let Content = [{
+Use ``__attribute__((cfi_salt("")))`` on a function declaration, function
---
https://github.com/tgymnich approved this pull request.
https://github.com/llvm/llvm-project/pull/146547
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
denzor200 wrote:
`constexpr` is one of the important additions of C++11. Most of the users will
search this check in "modernize" section and will be misled by not finding it
there.
"misc-const-correctness" - not a modernize check at all, it even relevant for
C++98, so it exists in "misc"
htt
https://github.com/erichkeane commented:
I'm not really a fan of how we're doing this. This should only be caused by a
failed lookup. This also seems about 2x as complicated as it needs to be. I
like the idea ehre, but not at this level of complexity.
@AaronBallman perhaps has a better idea
@@ -855,35 +857,24 @@ FileID
SourceManager::getFileIDLocal(SourceLocation::UIntTy SLocOffset) const {
break;
}
- NumProbes = 0;
- while (true) {
-unsigned MiddleIndex = (GreaterIndex-LessIndex)/2+LessIndex;
-SourceLocation::UIntTy MidOffset =
-getLoc
@@ -855,35 +857,24 @@ FileID
SourceManager::getFileIDLocal(SourceLocation::UIntTy SLocOffset) const {
break;
}
- NumProbes = 0;
- while (true) {
-unsigned MiddleIndex = (GreaterIndex-LessIndex)/2+LessIndex;
-SourceLocation::UIntTy MidOffset =
-getLoc
tromey wrote:
FWIW I read through the DWARF/debuginfo bits and it looked reasonable to me. I
found a couple small nits. I didn't read the other parts though.
https://github.com/llvm/llvm-project/pull/141937
___
cfe-commits mailing list
cfe-commits@l
@@ -118,8 +138,8 @@ class Document {
BulletList &addBulletList();
/// Doesn't contain any trailing newlines.
- /// We try to make the markdown human-readable, e.g. avoid extra escaping.
- /// At least one client (coc.nvim) displays the markdown verbatim!
Author: Kazu Hirata
Date: 2025-07-01T10:42:14-07:00
New Revision: 72a9c3cced898c3a8f7dcbbfaedca5e5438c9544
URL:
https://github.com/llvm/llvm-project/commit/72a9c3cced898c3a8f7dcbbfaedca5e5438c9544
DIFF:
https://github.com/llvm/llvm-project/commit/72a9c3cced898c3a8f7dcbbfaedca5e5438c9544.diff
L
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/146547
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AmrDeveloper created
https://github.com/llvm/llvm-project/pull/146577
Fix initalizing ValueRange with ArrayRef
`ValueRange(ArrayRef(std::forward(arg))) {}` warning
>From fc24ac864fe0517ab8c78969d4a508c1a9baf2af Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Tue, 1 Jul 20
llvmbot wrote:
@llvm/pr-subscribers-clangir
Author: Amr Hesham (AmrDeveloper)
Changes
Fix initalizing ValueRange with ArrayRef
`ValueRange(ArrayRef(std::forward(arg))) {}` warning
---
Full diff: https://github.com/llvm/llvm-project/pull/146577.diff
1 Files Affected:
- (mod
@@ -1203,6 +1204,8 @@ def : ProcessorModel<"cortex-x4", NeoverseV2Model,
ProcessorFeatures.X4,
[TuneX4]>;
def : ProcessorModel<"cortex-x925", NeoverseV2Model, ProcessorFeatures.X925,
[TuneX925]>;
+def : ProcessorModel<"gb10", NeoverseV
@@ -165,14 +165,17 @@ void Writer::createCustomSections() {
for (auto &pair : customSectionMapping) {
StringRef name = pair.first;
LLVM_DEBUG(dbgs() << "createCustomSection: " << name << "\n");
-
-OutputSection *sec = make(std::string(name), pair.second);
+Out
5chmidti wrote:
I ran this check on all of `clang/` and `clang-tools-extra/` and got no
compilation errors from too aggressively changing things to `constexpr`.
Some perf info from running the checks enabled in LLVM + this check:
Running on: `clang/lib/Sema/SemaExprCXX.cpp` (1 diag):
```text
=
https://github.com/5chmidti updated
https://github.com/llvm/llvm-project/pull/146553
>From 16d5a3fb942673d1fda0ef4b17363a0b5d634fbb Mon Sep 17 00:00:00 2001
From: Julian Schmidt
Date: Fri, 6 Sep 2024 22:58:46 +0200
Subject: [PATCH] [clang-tidy] add misc-constexpr check
This check finds all fun
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/146552
So we get the proper type for a heap-allocated value.
>From e8209bfa48669fc77e38ed9d20fd15d7795fdd35 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Tue, 1 Jul 2025 17:37:10 +0200
Subject: [
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Julian Schmidt (5chmidti)
Changes
This check finds all functions and variables that can be declared as
`constexpr`, using the specified standard version to check if the
requirements are met.
Fixes #115622
---
Patch is 95.12
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
So we get the proper type for a heap-allocated value.
---
Full diff: https://github.com/llvm/llvm-project/pull/146552.diff
2 Files Affected:
- (modified) clang/lib/AST/ByteCode/InterpBuiltinBitCast.cpp (+4
vbvictor wrote:
I'm +1 on the `modernize-use-constexpr`.
https://github.com/llvm/llvm-project/pull/146553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,562 @@
+// RUN: %check_clang_tidy -std=c++11 -check-suffix=11 %s misc-use-constexpr %t
-- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy -std=c++14 -check-suffix=11,14 %s misc-use-constexpr
%t -- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy
@@ -0,0 +1,946 @@
+//===--- UseConstexprCheck.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
@@ -19,6 +19,7 @@ set_target_properties(genconfusable PROPERTIES FOLDER "Clang
Tools Extra/Sourceg
add_clang_library(clangTidyMiscModule STATIC
ConstCorrectnessCheck.cpp
+ UseConstexprCheck.cpp
vbvictor wrote:
Please fix ordering
https://github.com/llvm/
@@ -0,0 +1,946 @@
+//===--- UseConstexprCheck.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,946 @@
+//===--- UseConstexprCheck.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
@@ -43,6 +44,7 @@ class MiscModule : public ClangTidyModule {
"misc-confusable-identifiers");
CheckFactories.registerCheck(
"misc-const-correctness");
+CheckFactories.registerCheck("misc-use-constexpr");
vbvictor wrote:
ditto
https://g
@@ -0,0 +1,946 @@
+//===--- UseConstexprCheck.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,946 @@
+//===--- UseConstexprCheck.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
@@ -463,6 +463,27 @@ class Triple {
const std::string &str() const { return Data; }
+ /// Return the triple string but only keep the first \p N components.
+ ///
+ /// The returned string will preserve the first \p N components exactly the
+ /// same as the original (in
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/145157
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/145157
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/royitaqi edited
https://github.com/llvm/llvm-project/pull/145157
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
5chmidti wrote:
> I tried with [ITK](https://github.com/InsightSoftwareConsortium/ITK/) and the
> resulting transformation did not compile.
>
> Many changes were of this form:
>
> ```diff
> -const double Max = 1.0 - Min;
> +constexpr double Max = 1.0 - Min;
> ```
>
> Which is
Author: Amr Hesham
Date: 2025-07-01T20:32:04+02:00
New Revision: 8dbfe83681ed2066b3591be24c4a566b87177d12
URL:
https://github.com/llvm/llvm-project/commit/8dbfe83681ed2066b3591be24c4a566b87177d12
DIFF:
https://github.com/llvm/llvm-project/commit/8dbfe83681ed2066b3591be24c4a566b87177d12.diff
LO
@@ -80,6 +81,11 @@ Parser::ParseStatementOrDeclaration(StmtVector &Stmts,
assert((CXX11Attrs.empty() || Res.isInvalid() || Res.isUsable()) &&
"attributes on empty statement");
+ if (HasStdAttr && getLangOpts().C99 &&
AaronBallman wrote:
Why are al
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/146577
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3639,6 +3639,64 @@ make the function's CFI jump table canonical. See
:ref:`the CFI documentation
}];
}
+def CFISaltDocs : Documentation {
+ let Category = DocCatFunction;
+ let Content = [{
+Use ``__attribute__((cfi_salt("")))`` on a function declaration, function
---
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/146283
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shiltian approved this pull request.
https://github.com/llvm/llvm-project/pull/146409
___
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 `ml-opt-rel-x86-64` running
on `ml-opt-rel-x86-64-b1` while building `clang,llvm` at step 6
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/185/builds/21236
Here i
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/146552
___
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: Timm Baeder (tbaederr)
Changes
This was only accidentally dropped, so add it back.
---
Full diff: https://github.com/llvm/llvm-project/pull/146589.diff
1 Files Affected:
- (modified) clang/lib/AST/ByteCode/Interp.h (+1-5)
``d
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/146204
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Amr Hesham
Date: 2025-07-01T20:59:53+02:00
New Revision: 6fafa46f8c1b335d08a59febb8bcb35a17fb177e
URL:
https://github.com/llvm/llvm-project/commit/6fafa46f8c1b335d08a59febb8bcb35a17fb177e
DIFF:
https://github.com/llvm/llvm-project/commit/6fafa46f8c1b335d08a59febb8bcb35a17fb177e.diff
LO
https://github.com/AmrDeveloper closed
https://github.com/llvm/llvm-project/pull/146239
___
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/146204
>From 57396eb457fb267a7b1ddfd79569dc3aa239535b Mon Sep 17 00:00:00 2001
From: AmrDeveloper
Date: Sat, 28 Jun 2025 10:35:06 +0200
Subject: [PATCH] [CIR] Comma Operator for ComplexType
---
clang/lib/CIR/Cod
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/146589
This was only accidentally dropped, so add it back.
>From 6cf4ca471adacf319acf1fb73e60f664540ac88e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Tue, 1 Jul 2025 20:46:18 +0200
Subject: [PA
5chmidti wrote:
Sorry for the silly docs mistakes, btw.
I've updated the docs, added replacement string options, and fixed the `const`
keyword removal.
https://github.com/llvm/llvm-project/pull/146553
___
cfe-commits mailing list
cfe-commits@lists.l
@@ -1181,8 +1183,9 @@ class Preprocessor {
public:
Preprocessor(const PreprocessorOptions &PPOpts, DiagnosticsEngine &diags,
- const LangOptions &LangOpts, SourceManager &SM,
- HeaderSearch &Headers, ModuleLoader &TheModuleLoader,
+
https://github.com/pizzud updated
https://github.com/llvm/llvm-project/pull/67467
>From 04a3e8d8cbd6943f44a81fddb0524902202a1a78 Mon Sep 17 00:00:00 2001
From: David Pizzuto
Date: Tue, 26 Sep 2023 10:45:42 -0700
Subject: [PATCH 01/21] [clang-tidy] Add bugprone-move-shared-pointer-contents
chec
@@ -0,0 +1,562 @@
+// RUN: %check_clang_tidy -std=c++11 -check-suffix=11 %s misc-use-constexpr %t
-- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy -std=c++14 -check-suffix=11,14 %s misc-use-constexpr
%t -- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy
@@ -165,14 +165,17 @@ void Writer::createCustomSections() {
for (auto &pair : customSectionMapping) {
StringRef name = pair.first;
LLVM_DEBUG(dbgs() << "createCustomSection: " << name << "\n");
-
-OutputSection *sec = make(std::string(name), pair.second);
+Out
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Author: Kazu Hirata (kazutakahirata)
Changes
ME is already of ObjCMessageExpr *.
---
Full diff: https://github.com/llvm/llvm-project/pull/146547.diff
1 Files Affected:
- (modified) clang/lib/Analysis/CFG.cpp (+1-2)
``diff
d
@@ -3613,6 +3613,16 @@ class Sema final : public SemaBase {
ParsedType &SuggestedType,
bool IsTemplateName = false);
+ // Try to suggest the missing standard library include files
+ //
+ // \param SymbolName the
@@ -554,6 +563,11 @@ bool Parser::ParseOptionalCXXScopeSpecifier(
break;
}
+ if (FailedNesatedNameBuilding && Tok.getKind() == tok::identifier) {
Mr-Anyone wrote:
I completed agree with you for the fact that this is really ugly code here.
The problem
https://github.com/yuri91 edited
https://github.com/llvm/llvm-project/pull/146230
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -179,22 +179,39 @@ StringRef sys::detail::getHostCPUNameForARM(StringRef
ProcCpuinfoContent) {
// Look for the CPU implementer line.
StringRef Implementer;
StringRef Hardware;
- StringRef Part;
+ SmallVector Parts;
for (unsigned I = 0, E = Lines.size(); I != E; +
https://github.com/kazutakahirata created
https://github.com/llvm/llvm-project/pull/146547
ME is already of ObjCMessageExpr *.
>From e0f784569b26f2a8dd52fb73c1fd142f2d37ca1b Mon Sep 17 00:00:00 2001
From: Kazu Hirata
Date: Tue, 1 Jul 2025 07:49:58 -0700
Subject: [PATCH] [Analysis] Remove an u
carlosgalvezp wrote:
misc-use-constexpr? :)
https://github.com/llvm/llvm-project/pull/146553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,562 @@
+// RUN: %check_clang_tidy -std=c++11 -check-suffix=11 %s misc-use-constexpr %t
-- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy -std=c++14 -check-suffix=11,14 %s misc-use-constexpr
%t -- -- -fno-delayed-template-parsing
+// RUN: %check_clang_tidy
@@ -2344,11 +2344,15 @@ void ModuleBitcodeWriter::writeDILocalVariable(
void ModuleBitcodeWriter::writeDILabel(
const DILabel *N, SmallVectorImpl &Record,
unsigned Abbrev) {
- Record.push_back((uint64_t)N->isDistinct());
+ uint64_t IsArtificialFlag = uint64_t(N->isArt
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Daniel Paoliello (dpaoliello)
Changes
There is no way in Arm64 Windows to indicate that a given function has used the
Frame Pointer as a General Purpose Register, as such stack walks will always
assume that the frame chain is valid
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Daniel Paoliello (dpaoliello)
Changes
There is no way in Arm64 Windows to indicate that a given function has used the
Frame Pointer as a General Purpose Register, as such stack walks will always
assume that the frame chain is valid and wi
carlosgalvezp wrote:
I'm ok with modernize-use-constexpr as well!
https://github.com/llvm/llvm-project/pull/146553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,148 @@
+; Tests that we add DILabels for the suspend points.
+;
+; We check both the generated LLVM:
+; RUN: opt < %s -passes='cgscc(coro-split)' -S | FileCheck %s
+;
+; And the debug info:
+; RUN: opt < %s -passes='cgscc(coro-split),coro-cleanup' \
+; RUN: | llc -O0
@@ -1478,6 +1479,18 @@ struct SwitchCoroutineSplitter {
static void createResumeEntryBlock(Function &F, coro::Shape &Shape) {
LLVMContext &C = F.getContext();
+DIBuilder DBuilder(*F.getParent(), /*AllowUnresolved*/ false);
+DISubprogram *DIS = F.getSubprogram();
@@ -6975,16 +6975,22 @@ appear in the included source file.
DILabel
"""
-``DILabel`` nodes represent labels within a :ref:`DISubprogram`. All fields of
-a ``DILabel`` are mandatory. The ``scope:`` field must be one of either a
-:ref:`DILexicalBlockFile`, a :ref:`DILexicalB
@@ -0,0 +1,148 @@
+; Tests that we add DILabels for the suspend points.
+;
+; We check both the generated LLVM:
+; RUN: opt < %s -passes='cgscc(coro-split)' -S | FileCheck %s
+;
+; And the debug info:
+; RUN: opt < %s -passes='cgscc(coro-split),coro-cleanup' \
+; RUN: | llc -O0
@@ -2240,14 +2242,28 @@ Error
MetadataLoader::MetadataLoaderImpl::parseOneMetadata(
break;
}
case bitc::METADATA_LABEL: {
-if (Record.size() != 5)
+if (Record.size() < 5 || Record.size() > 7)
return error("Invalid record");
IsDistinct = Record[0]
@@ -688,9 +688,9 @@ static DIType *solveDIType(DIBuilder &Builder, Type *Ty,
static void buildFrameDebugInfo(Function &F, coro::Shape &Shape,
FrameDataInfo &FrameData) {
DISubprogram *DIS = F.getSubprogram();
- // If there is no DISubprogram f
1 - 100 of 443 matches
Mail list logo
