https://github.com/kito-cheng approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/141007
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/140147
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Finn Plummer
Date: 2025-05-21T14:40:13-07:00
New Revision: b499f7f2b246c284f8deb428f68d0b488b80c630
URL:
https://github.com/llvm/llvm-project/commit/b499f7f2b246c284f8deb428f68d0b488b80c630
DIFF:
https://github.com/llvm/llvm-project/commit/b499f7f2b246c284f8deb428f68d0b488b80c630.diff
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`ppc64le-lld-multistage-test` running on `ppc64le-lld-multistage-test` while
building `clang,llvm` at step 6 "build-stage1-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/168/builds/12321
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/140148
>From ae3cffb72588f7afa5107dbd0b7d47137336273a Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Thu, 15 May 2025 19:49:44 +
Subject: [PATCH 1/2] [HLSL][RootSignature] Add parsing of Register in params
for
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/136828
>From 927380bdf377581e6e40507a12c374023352c613 Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Wed, 30 Apr 2025 22:26:59 -0700
Subject: [PATCH 1/2] [clang][PAC] add support for options parameter to
__ptrauth
T
@@ -948,6 +950,165 @@ void CIRGenFunction::emitIgnoredExpr(const Expr *e) {
emitLValue(e);
}
+// Handle the case where the condition is a constant evaluatable simple
integer,
+// which means we don't have to separately handle the true/false blocks.
+static std::optional han
https://github.com/bd1976bris updated
https://github.com/llvm/llvm-project/pull/140956
>From 0d9218eda0b99b0d439afb8a84a056581362638e Mon Sep 17 00:00:00 2001
From: Dunbobbin
Date: Mon, 19 May 2025 10:17:00 +0100
Subject: [PATCH 1/7] [Windows][LLVM] Elide `PrettyStackTrace` output for usage
er
https://github.com/qinkunbao edited
https://github.com/llvm/llvm-project/pull/140964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bd1976bris edited
https://github.com/llvm/llvm-project/pull/140956
___
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: Qinkun Bao (qinkunbao)
Changes
As discussed in https://github.com/llvm/llvm-project/pull/139772,
Matcher::Globs can keep the order when parsing the case list.
---
Full diff: https://github.com/llvm/llvm-project/pull/140964.diff
3 Files
https://github.com/qinkunbao converted_to_draft
https://github.com/llvm/llvm-project/pull/140964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
bd1976bris wrote:
I wonder if we need a release note for this behaviour change? I've added on in
case.
https://github.com/llvm/llvm-project/pull/140956
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
https://github.com/Mr-Anyone updated
https://github.com/llvm/llvm-project/pull/140278
>From 3945cae10c7f2629e7e875db29bd9aa4fa74055c Mon Sep 17 00:00:00 2001
From: Vincent
Date: Wed, 14 May 2025 16:36:35 -0400
Subject: [PATCH 1/5] [clang] Fixed Constant Evaluation don't Call Destructor
Within
https://github.com/ojhunt updated
https://github.com/llvm/llvm-project/pull/136828
>From 927380bdf377581e6e40507a12c374023352c613 Mon Sep 17 00:00:00 2001
From: Oliver Hunt
Date: Wed, 30 Apr 2025 22:26:59 -0700
Subject: [PATCH] [clang][PAC] add support for options parameter to __ptrauth
This P
@@ -6009,6 +6009,8 @@ def note_not_found_by_two_phase_lookup : Note<"%0 should
be declared prior to th
def err_undeclared_use : Error<"use of undeclared %0">;
def warn_deprecated : Warning<"%0 is deprecated">,
InGroup;
+def warn_deprecated_switch_case : Warning<"%0 is depr
@@ -61,11 +61,14 @@ struct ScopedFatalErrorHandler {
/// @deprecated Use reportFatalInternalError() or reportFatalUsageError()
/// instead.
[[noreturn]] LLVM_ABI void report_fatal_error(const char *reason,
- bool gen_crash_diag = tru
@@ -1633,8 +1633,8 @@ static void repeatForLambdaConversionFunctionCallingConvs(
CC_C,CC_X86StdCall, CC_X86FastCall, CC_X86VectorCall,
DefaultFree, DefaultMember, CallOpCC};
llvm::sort(Convs);
-llvm::iterator_range Range(
-std::begin(Conv
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-m68k-linux-cross`
running on `suse-gary-m68k-cross` while building `clang,llvm` at step 4 "build
stage 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/27/builds/10379
Here is the relevant p
@@ -1633,8 +1633,8 @@ static void repeatForLambdaConversionFunctionCallingConvs(
CC_C,CC_X86StdCall, CC_X86FastCall, CC_X86VectorCall,
DefaultFree, DefaultMember, CallOpCC};
llvm::sort(Convs);
-llvm::iterator_range Range(
-std::begin(Conv
@@ -2507,13 +2441,9 @@ bool SemaHLSL::CheckBuiltinFunctionCall(unsigned
BuiltinID, CallExpr *TheCall) {
break;
}
case Builtin::BI__builtin_hlsl_dot: {
-if (SemaRef.checkArgCount(TheCall, 2))
farzonl wrote:
a comment to this effect would be good, b
https://github.com/Mr-Anyone updated
https://github.com/llvm/llvm-project/pull/140278
>From 3945cae10c7f2629e7e875db29bd9aa4fa74055c Mon Sep 17 00:00:00 2001
From: Vincent
Date: Wed, 14 May 2025 16:36:35 -0400
Subject: [PATCH 1/5] [clang] Fixed Constant Evaluation don't Call Destructor
Within
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/140629
>From 5b8f0635fdddc87ea326a3b1b47191ab9bf23c0f Mon Sep 17 00:00:00 2001
From: Oleksandr Tarasiuk
Date: Tue, 20 May 2025 00:38:43 +0300
Subject: [PATCH 1/4] [Clang] add typo correction for unknown attribute na
https://github.com/perry-ca updated
https://github.com/llvm/llvm-project/pull/140944
>From 7e3ff9fbe877ab54fd16bd41e26f32798f0ab430 Mon Sep 17 00:00:00 2001
From: Sean Perry
Date: Wed, 21 May 2025 18:02:26 +
Subject: [PATCH 1/2] Implement _Export keyword
---
clang/docs/ReleaseNotes.rst
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `flang-aarch64-latest-gcc`
running on `linaro-flang-aarch64-latest-gcc` while building `clang,llvm` at
step 5 "build-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/130/builds/13279
Her
https://github.com/bd1976bris updated
https://github.com/llvm/llvm-project/pull/140956
>From 0d9218eda0b99b0d439afb8a84a056581362638e Mon Sep 17 00:00:00 2001
From: Dunbobbin
Date: Mon, 19 May 2025 10:17:00 +0100
Subject: [PATCH 1/4] [Windows][LLVM] Elide `PrettyStackTrace` output for usage
er
@@ -1633,8 +1633,8 @@ static void repeatForLambdaConversionFunctionCallingConvs(
CC_C,CC_X86StdCall, CC_X86FastCall, CC_X86VectorCall,
DefaultFree, DefaultMember, CallOpCC};
llvm::sort(Convs);
-llvm::iterator_range Range(
-std::begin(Conv
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/140148
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/bd1976bris updated
https://github.com/llvm/llvm-project/pull/140956
>From 0d9218eda0b99b0d439afb8a84a056581362638e Mon Sep 17 00:00:00 2001
From: Dunbobbin
Date: Mon, 19 May 2025 10:17:00 +0100
Subject: [PATCH 1/6] [Windows][LLVM] Elide `PrettyStackTrace` output for usage
er
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/140148
>From ae3cffb72588f7afa5107dbd0b7d47137336273a Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Thu, 15 May 2025 19:49:44 +
Subject: [PATCH 1/3] [HLSL][RootSignature] Add parsing of Register in params
for
https://github.com/qinkunbao created
https://github.com/llvm/llvm-project/pull/140964
As discussed in https://github.com/llvm/llvm-project/pull/139772,
Matcher::Globs can keep the order when parsing the case list.
>From 534d49d489476ffd1aa8f23d21a02f29d3b823fc Mon Sep 17 00:00:00 2001
From: Q
https://github.com/bd1976bris updated
https://github.com/llvm/llvm-project/pull/140956
>From 23b762fa8adae76ffe7e85a328aef8710030d831 Mon Sep 17 00:00:00 2001
From: Dunbobbin
Date: Mon, 19 May 2025 10:17:00 +0100
Subject: [PATCH 1/2] [LLVM][Windows] Elide `PrettyStackTrace` output for usage
er
vsapsai wrote:
> @vsapsai I guess, it's a good sign? :) Do you see how our use case can be
> supported by a trivial and low-risk forward fix? If not, I'd insist on a
> revert before we can figure out the way forward. We can run this sort of a
> change through our testing before it relands, and
zyn0217 wrote:
I feel an invalid Decl shouldn't make its way down so deeper into
instantiation. Could you check?
https://github.com/llvm/llvm-project/pull/140905
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
Author: Balázs Benics
Date: 2025-05-21T13:33:51+02:00
New Revision: 881872896279338788690ba3f17b6c756ae4b91c
URL:
https://github.com/llvm/llvm-project/commit/881872896279338788690ba3f17b6c756ae4b91c
DIFF:
https://github.com/llvm/llvm-project/commit/881872896279338788690ba3f17b6c756ae4b91c.diff
@@ -7639,6 +7639,8 @@ def warn_param_mismatched_alignment : Warning<
def err_objc_object_assignment : Error<
"cannot assign to class object (%0 invalid)">;
+def err_typecheck_array_prvalue_operand : Error<
+ "array prvalue is not permitted">;
AaronBallman w
https://github.com/balazs-benics-sonarsource closed
https://github.com/llvm/llvm-project/pull/140861
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
> @AaronBallman
>
> I'm not sure who's responsible for `` in LLVM.
Yeah, it's never really clear who is expected to provide these headers.
> In GCC it's a header provided by the compiler. I see that this PR does not
> add the header, so I was wondering if that's to be adde
@@ -28,25 +30,39 @@ int bar(C *c) {
int bar2(C *c) {
// CHECK-NEW-LABEL: _Z4bar2P1C
-// CHECK-NEW: load i32, {{.*}}, !tbaa [[TAG_int:!.*]]
+// CHECK-NEW: load i32, {{.*}}, !tbaa [[TAG_C_x:!.*]]
return c->x[2];
}
int bar3(C *c, int j) {
// CHECK-NEW-LABEL: _Z4bar3P1Ci
-
Author: Sebastian Pop
Date: 2025-05-21T08:27:39-05:00
New Revision: d36028120a6ef6346bfaafe82d4d1a2887cf5e33
URL:
https://github.com/llvm/llvm-project/commit/d36028120a6ef6346bfaafe82d4d1a2887cf5e33
DIFF:
https://github.com/llvm/llvm-project/commit/d36028120a6ef6346bfaafe82d4d1a2887cf5e33.diff
https://github.com/sebpop closed
https://github.com/llvm/llvm-project/pull/140182
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DanielCChen edited
https://github.com/llvm/llvm-project/pull/140850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -935,6 +935,14 @@ ToolChain::getTargetSubDirPath(StringRef BaseDir) const {
if (auto Path = getPathForTriple(T))
return *Path;
+ // On AIX, the environment component is not used in the target sub dir name.
DanielCChen wrote:
We already have a simila
https://github.com/AaronBallman created
https://github.com/llvm/llvm-project/pull/140890
WG14 N3469 changed _Lengthof to _Countof but it also introduced the
header to expose a macro with a non-ugly identifier. GCC vends
this header as part of the compiler implementation, so Clang should do th
https://github.com/dmpolukhin updated
https://github.com/llvm/llvm-project/pull/140912
>From e3ce9bb18f165649b00db14b2282649315b28883 Mon Sep 17 00:00:00 2001
From: Dmitry Polukhin
Date: Wed, 21 May 2025 07:54:02 -0700
Subject: [PATCH 1/2] [clang-tidy][performance-unnecessary-value-param] Avoid
https://github.com/mshockwave approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/140681
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jmorse approved this pull request.
Much clearer, LGTM thanks!
https://github.com/llvm/llvm-project/pull/134632
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -8811,6 +8820,68 @@ void AMDGPUAsmParser::cvtVINTERP(MCInst &Inst, const
OperandVector &Operands)
Inst.getOperand(ModIdx).setImm(ModVal);
}
}
+void AMDGPUAsmParser::cvtScaledMFMA(MCInst &Inst,
+const OperandVector &Operands) {
+ Op
sivadeilra wrote:
@efriedma-quic - I'm looking at several approaches for handling this, but
before I go too far with it, it would help to know just how general or complete
the solution needs to be. The problem is that the transformation I'm doing
will necessarily change a constant expression
https://github.com/balazs-benics-sonarsource created
https://github.com/llvm/llvm-project/pull/140924
Tranersing the CFG blocks of a function is a fundamental operation. Many C++
constructs can create splits in the control-flow, such as `if`, `for`, and
similar control structures or ternary ex
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balázs Benics (balazs-benics-sonarsource)
Changes
Tranersing the CFG blocks of a function is a fundamental operation. Many C++
constructs can create splits in the control-flow, such as `if`, `for`, and
similar control st
https://github.com/AaronBallman created
https://github.com/llvm/llvm-project/pull/140925
There are two related issues being fixed in this patch. Both issues relate to
use of an invalid structure which contains a member that we error recover such
that the field has the same type as the structur
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/137183
___
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: Aaron Ballman (AaronBallman)
Changes
There are two related issues being fixed in this patch. Both issues relate to
use of an invalid structure which contains a member that we error recover such
that the field has the same type as the stru
https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/140183
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -8811,6 +8820,68 @@ void AMDGPUAsmParser::cvtVINTERP(MCInst &Inst, const
OperandVector &Operands)
Inst.getOperand(ModIdx).setImm(ModVal);
}
}
+void AMDGPUAsmParser::cvtScaledMFMA(MCInst &Inst,
+const OperandVector &Operands) {
+ Op
@@ -8811,6 +8820,68 @@ void AMDGPUAsmParser::cvtVINTERP(MCInst &Inst, const
OperandVector &Operands)
Inst.getOperand(ModIdx).setImm(ModVal);
}
}
+void AMDGPUAsmParser::cvtScaledMFMA(MCInst &Inst,
+const OperandVector &Operands) {
+ Op
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/140925
___
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/140828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/140828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,81 @@
+// RUN: %check_clang_tidy -std=c++20-or-later %s
performance-unnecessary-value-param %t -- -fix-errors
+
+namespace std {
+
+template struct coroutine_traits {
+ using promise_type = typename R::promise_type;
+};
+
+template struct coroutine_handle;
+
+templa
https://github.com/Andres-Salamanca updated
https://github.com/llvm/llvm-project/pull/140649
>From c1403f148a58e259cc296310dc21b8c5611f2e82 Mon Sep 17 00:00:00 2001
From: Andres Salamanca
Date: Mon, 19 May 2025 18:53:15 -0500
Subject: [PATCH 1/3] Implement CIR switch case simplify with appropri
DanielCChen wrote:
> A more general question:
> It seems neither `compile-rt` nor `flang-rt` uses `getTargetSubDirPath`, so
> we didn't need to strip to the "base" triple.
> Sanitizer uses it apparently. Is it by design?
The reason we didn't need to change `getTargetSubDirPath` is because we st
dwblaikie wrote:
yeah, can't say I have any recollection of why function declaration would be in
a "retained types" list - probably incidental/not especially intentional (we
didn't make declarations for functions, except member functions, until
"recently" when call site debug info was added).
https://github.com/languagelawyer edited
https://github.com/llvm/llvm-project/pull/140702
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,12 @@
+// RUN: %clang_cc1 -triple riscv64 -ast-dump -ast-dump-filter c23 -std=c23 -x
c %s | FileCheck --strict-whitespace %s
+
+// CHECK: FunctionDecl{{.*}}pre_c23
+// CHECK-NEXT:|-CompoundStmt
+// CHECK-NEXT:`-RISCVInterruptAttr{{.*}}supervisor
+__attrib
https://github.com/alejandro-colomar approved this pull request.
Thanks!
For the first commit:
```
Suggested-by: Alejandro Colomar
```
https://github.com/llvm/llvm-project/pull/140890
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://li
@@ -4898,6 +4898,234 @@ void
CGOpenMPRuntime::emitSingleReductionCombiner(CodeGenFunction &CGF,
}
}
+void CGOpenMPRuntime::emitPrivateReduction(
+CodeGenFunction &CGF, SourceLocation Loc, const Expr *Privates,
+const Expr *LHSExprs, const Expr *RHSExprs, const Expr
@@ -260,6 +260,7 @@ static bool isBuiltinHeaderName(StringRef FileName) {
.Case("stdarg.h", true)
.Case("stdatomic.h", true)
.Case("stdbool.h", true)
+ .Case("stdcountof.h", true)
erichkeane wrote:
These too!
https:/
https://github.com/jakeegan updated
https://github.com/llvm/llvm-project/pull/140850
>From 120423abadbfd63dbb50387fb5ce26ff0f3ff257 Mon Sep 17 00:00:00 2001
From: Jake Egan
Date: Wed, 21 May 2025 01:51:00 -0400
Subject: [PATCH 1/2] Handle triple environment component for target runtime
directo
Author: Alex MacLean
Date: 2025-05-21T08:14:15-07:00
New Revision: 735209c0688b10a66c24750422b35d8c2ad01bb5
URL:
https://github.com/llvm/llvm-project/commit/735209c0688b10a66c24750422b35d8c2ad01bb5
DIFF:
https://github.com/llvm/llvm-project/commit/735209c0688b10a66c24750422b35d8c2ad01bb5.diff
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `fuchsia-x86_64-linux`
running on `fuchsia-debian-64-us-central1-a-1` while building
`clang-tools-extra` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/11/builds/15608
Here is t
https://github.com/AlexMaclean closed
https://github.com/llvm/llvm-project/pull/140615
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4898,6 +4898,234 @@ void
CGOpenMPRuntime::emitSingleReductionCombiner(CodeGenFunction &CGF,
}
}
+void CGOpenMPRuntime::emitPrivateReduction(
+CodeGenFunction &CGF, SourceLocation Loc, const Expr *Privates,
+const Expr *LHSExprs, const Expr *RHSExprs, const Expr
@@ -4898,6 +4898,234 @@ void
CGOpenMPRuntime::emitSingleReductionCombiner(CodeGenFunction &CGF,
}
}
+void CGOpenMPRuntime::emitPrivateReduction(
+CodeGenFunction &CGF, SourceLocation Loc, const Expr *Privates,
+const Expr *LHSExprs, const Expr *RHSExprs, const Expr
https://github.com/balazske created
https://github.com/llvm/llvm-project/pull/140913
It became necessary to add `Sema` as dependency of `ASTImporter`.
This is needed to implement import of concept related nodes.
`Sema` can not be a dependency of `AST`, but this split of
`ASTImporter` makes it po
@@ -4898,6 +4898,234 @@ void
CGOpenMPRuntime::emitSingleReductionCombiner(CodeGenFunction &CGF,
}
}
+void CGOpenMPRuntime::emitPrivateReduction(
+CodeGenFunction &CGF, SourceLocation Loc, const Expr *Privates,
+const Expr *LHSExprs, const Expr *RHSExprs, const Expr
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Dmitry Polukhin (dmpolukhin)
Changes
Summary:
Replacing by-value parameters with passing by-reference is not safe for
coroutines because the caller may be executed in parallel with the callee,
which increases the chances of re
https://github.com/dmpolukhin created
https://github.com/llvm/llvm-project/pull/140912
Summary:
Replacing by-value parameters with passing by-reference is not safe for
coroutines because the caller may be executed in parallel with the callee,
which increases the chances of resulting in danglin
https://github.com/Mr-Anyone updated
https://github.com/llvm/llvm-project/pull/140828
>From 393ccc154e2f1d68ef25d660c8063ab9674ab09b Mon Sep 17 00:00:00 2001
From: Vincent
Date: Tue, 20 May 2025 21:25:32 -0400
Subject: [PATCH 1/5] [clang][TableGen] Fix Duplicate Entries in TableGen
Fixed Table
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Dmitry Polukhin (dmpolukhin)
Changes
Summary:
Replacing by-value parameters with passing by-reference is not safe for
coroutines because the caller may be executed in parallel with the callee,
which increases the chances of resulting
@@ -4898,6 +4898,274 @@ void
CGOpenMPRuntime::emitSingleReductionCombiner(CodeGenFunction &CGF,
}
}
+void CGOpenMPRuntime::emitPrivateReduction(
+CodeGenFunction &CGF, SourceLocation Loc, const Expr *Privates,
+const Expr *LHSExprs, const Expr *RHSExprs, const Expr
https://github.com/Mr-Anyone edited
https://github.com/llvm/llvm-project/pull/140828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DanielCChen wrote:
A more general question:
It seems neither `compile-rt` nor `flang-rt` uses `getTargetSubDirPath`, so we
didn't need to strip to the "base" triple.
Sanitizer uses it apparently. Is it by design? @daltenty
https://github.com/llvm/llvm-project/pull/140850
__
https://github.com/erichkeane approved this pull request.
https://github.com/llvm/llvm-project/pull/140828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DanielCChen deleted
https://github.com/llvm/llvm-project/pull/140850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Fraser Cormack
Date: 2025-05-21T17:50:00+01:00
New Revision: 94142d9bb08c523802cbfb3c316fc2ae9bb33b60
URL:
https://github.com/llvm/llvm-project/commit/94142d9bb08c523802cbfb3c316fc2ae9bb33b60
DIFF:
https://github.com/llvm/llvm-project/commit/94142d9bb08c523802cbfb3c316fc2ae9bb33b60.diff
@@ -935,6 +935,14 @@ ToolChain::getTargetSubDirPath(StringRef BaseDir) const {
if (auto Path = getPathForTriple(T))
return *Path;
+ // On AIX, the environment component is not used in the target sub dir name.
DanielCChen wrote:
I think it is correct th
@@ -16,6 +16,10 @@
// RUN:-resource-dir=%S/Inputs/resource_dir_with_per_target_subdir\
// RUN: | FileCheck --check-prefix=PRINT-RUNTIME-DIR64-PER-TARGET %s
+// RUN: %clang -print-runtime-dir --target=powerpc-ibm-aix-unknown \
+// RUN:-resource-dir=%S/Input
https://github.com/DanielCChen edited
https://github.com/llvm/llvm-project/pull/140850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DanielCChen deleted
https://github.com/llvm/llvm-project/pull/140850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DanielCChen edited
https://github.com/llvm/llvm-project/pull/140850
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -935,6 +935,14 @@ ToolChain::getTargetSubDirPath(StringRef BaseDir) const {
if (auto Path = getPathForTriple(T))
return *Path;
+ // On AIX, the environment component is not used in the target sub dir name.
jakeegan wrote:
It returns an `EnvironmentT
alejandro-colomar wrote:
Hi!
I don't see it mentioned, so I'll note it down here. The standard allows this:
```c
typedef typeof(sizeof(0)) size_t;
```
as a valid implementation of `size_t`.
It would be good to keep this in mind to not accidentally break it. (Not
saying this would break it
@@ -3727,12 +3734,26 @@ static void GenerateHasAttrSpellingStringSwitch(
}
}
-std::string TestStr = !Test.empty()
- ? Test + " ? " + itostr(Version) + " : 0"
- : itostr(Version);
-if (Scope.empty() ||
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/140828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3727,12 +3734,26 @@ static void GenerateHasAttrSpellingStringSwitch(
}
}
-std::string TestStr = !Test.empty()
- ? Test + " ? " + itostr(Version) + " : 0"
- : itostr(Version);
-if (Scope.empty() ||
@@ -3727,12 +3734,26 @@ static void GenerateHasAttrSpellingStringSwitch(
}
}
-std::string TestStr = !Test.empty()
- ? Test + " ? " + itostr(Version) + " : 0"
- : itostr(Version);
-if (Scope.empty() ||
https://github.com/AaronBallman commented:
Thank you for working on this! In general, this looks good to me, just nits
about our coding style.
https://github.com/llvm/llvm-project/pull/140828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
https://github.com/qinkunbao updated
https://github.com/llvm/llvm-project/pull/140529
>From b83755d2aa0c5417ab8f359aa842449213437a7a Mon Sep 17 00:00:00 2001
From: Qinkun Bao
Date: Mon, 19 May 2025 11:14:01 +
Subject: [PATCH 1/9] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UT
301 - 400 of 498 matches
Mail list logo
