Author: Fangrui Song
Date: 2024-03-18T13:11:44-07:00
New Revision: 9936ac308332796bf16024ce30a4aa1fba2cce66
URL:
https://github.com/llvm/llvm-project/commit/9936ac308332796bf16024ce30a4aa1fba2cce66
DIFF:
https://github.com/llvm/llvm-project/commit/9936ac308332796bf16024ce30a4aa1fba2cce66.diff
huangjd wrote:
My concern to enable this feature by default is that there are some less known
syntax (probably from other supported languages or extensions) that has a
MemberExpr but actually emitting something unexpected, which could cause a
crash. I am testing this feature on some real-life
gigaroby wrote:
I think the documentation failure is related to
https://github.com/llvm/llvm-project/commit/0c423af59c971ddf1aa12d94529edf8293608157
but I might be wrong.
https://github.com/llvm/llvm-project/pull/84346
___
cfe-commits mailing list
@@ -8,8 +8,8 @@ struct S0 {
// CHECK-LABEL: define dso_local void @_Z3fS0v
// CHECK:alloca %struct.S0, align 4
// CHECK-NEXT:[[TMP0:%.*]] = alloca %struct.S0, align 4
-// CHECK: call void @llvm.dbg.declare(metadata ptr [[TMP0]], metadata
https://github.com/adrian-prantl edited
https://github.com/llvm/llvm-project/pull/85665
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/adrian-prantl requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/85665
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ributzka approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/85348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-coroutines
Author: Yuxuan Chen (yuxuanchen1997)
Changes
This patch aims to fix https://github.com/llvm/llvm-project/issues/77111
The original issue is crash-on-invalid. However, Clang currently accepts
programs where `await_suspend` returns a type
https://github.com/yuxuanchen1997 created
https://github.com/llvm/llvm-project/pull/85684
This patch aims to fix https://github.com/llvm/llvm-project/issues/77111
The original issue is crash-on-invalid. However, Clang currently accepts
programs where `await_suspend` returns a type that's
https://github.com/alexcrichton updated
https://github.com/llvm/llvm-project/pull/84569
>From 99ca952b60344d2ff683d05d8baa423aa11da83a Mon Sep 17 00:00:00 2001
From: Alex Crichton
Date: Fri, 8 Mar 2024 13:36:18 -0800
Subject: [PATCH 1/2] [WebAssembly] Change the default linker for
dwblaikie wrote:
Reading LLVM IR lit CHECK lines from clang codegen is a bit difficult - could
you include some simple examples (perhaps from the new clang tests in this
patch) showing the DWARF output just as comments in this review for something
more easily glanceable?
As for flags - I
https://github.com/sunfishcode approved this pull request.
Looks good!
https://github.com/llvm/llvm-project/pull/84569
___
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 7d55b916a5e5091ff21d6fea4aaa54efe73535a7
d1a1b22da3e1361a34f2d4fa8a2f9ef361f109ee --
alexcrichton wrote:
At @sunfishcode's request I've pushed up a second commit which pass the path to
`wasm-ld` to `wasm-component-ld`, and in testing it I also added the ability
for `-fuse-ld=lld` to explicitly request that `wasm-ld` is used, regardless of
target.
https://github.com/fhahn updated https://github.com/llvm/llvm-project/pull/84569
>From 99ca952b60344d2ff683d05d8baa423aa11da83a Mon Sep 17 00:00:00 2001
From: Alex Crichton
Date: Fri, 8 Mar 2024 13:36:18 -0800
Subject: [PATCH 1/2] [WebAssembly] Change the default linker for
`wasm32-wasip2`
https://github.com/alexcrichton updated
https://github.com/llvm/llvm-project/pull/84569
>From 99ca952b60344d2ff683d05d8baa423aa11da83a Mon Sep 17 00:00:00 2001
From: Alex Crichton
Date: Fri, 8 Mar 2024 13:36:18 -0800
Subject: [PATCH 1/2] [WebAssembly] Change the default linker for
https://github.com/jdoerfert approved this pull request.
LG.
https://github.com/llvm/llvm-project/pull/85363
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Prabhuk approved this pull request.
https://github.com/llvm/llvm-project/pull/84146
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Prabhuk wrote:
Verified that the reproducer for the original crash fails before the revert
patch:
https://github.com/llvm/llvm-project/pull/82032/commits/13bf4429ba2e33f9069f632c974d8fdf121acefa
and toolchain built with current patch *does not fail* against the reproducer.
Though I haven't
https://github.com/zahiraam updated
https://github.com/llvm/llvm-project/pull/84014
>From bdefe754c14c5e050ebf2b9c82eca458041564a4 Mon Sep 17 00:00:00 2001
From: Zahira Ammarguellat
Date: Tue, 5 Mar 2024 05:35:16 -0800
Subject: [PATCH 01/10] [clang-cl] Fix value of __FUNCTION__ in MSVC mode.
@@ -129,19 +215,19 @@ auto inPlaceClass() {
auto isOptionalNulloptConstructor() {
return cxxConstructExpr(
- hasOptionalType(),
+ hasOptionalOrDerivedType(),
ymand wrote:
Here and below -- now that this matcher is more expensive, please move
zahiraam wrote:
I have uploaded your patch and compared the attributes generated from your
patch and the little test case from
https://github.com/llvm/llvm-project/issues/62098. The attributes generated
are different. Therefore, the expected optimizations from this patch are going
to be
@@ -64,39 +64,117 @@ static bool hasOptionalClassName(const CXXRecordDecl ) {
return false;
}
+static const CXXRecordDecl *getOptionalBaseClass(const CXXRecordDecl *RD) {
+ if (RD == nullptr)
+return nullptr;
+ if (hasOptionalClassName(*RD))
+return RD;
+
+ if
https://github.com/ymand approved this pull request.
https://github.com/llvm/llvm-project/pull/84138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -119,20 +119,28 @@ QualType getPublicType(const Expr *E) {
return Ty;
}
- QualType Ty = getPublicType(Cast->getSubExpr());
-
- // Is `Ty` the type of `*this`? In this special case, we can upcast to the
- // base class even if the base is non-public.
- bool
https://github.com/ymand edited https://github.com/llvm/llvm-project/pull/84138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ymand approved this pull request.
thanks!
https://github.com/llvm/llvm-project/pull/85623
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Prabhuk wrote:
Apologies that this had slipped out of my radar. I am trying this patch now
agains the reproducer from the original failure. I'll update the results here
in a few hours. Thank you.
https://github.com/llvm/llvm-project/pull/84146
___
@@ -855,7 +865,14 @@ void DeclPrinter::VisitFunctionDecl(FunctionDecl *D) {
Out << Proto << " -> ";
Proto.clear();
}
- AFT->getReturnType().print(Out, Policy, Proto);
+ if (!Policy.SuppressTagKeyword && Policy.SuppressScope &&
+
@@ -1022,7 +1039,13 @@ void DeclPrinter::VisitVarDecl(VarDecl *D) {
? D->getIdentifier()->deuglifiedName()
: D->getName();
- printDeclType(T, Name);
+ if (!Policy.SuppressTagKeyword && Policy.SuppressScope &&
+ !Policy.SuppressUnwrittenScope)
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/84014
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
I spotted some simplifications that can be applied. If precommit CI comes back
green, then I think this is basically ready to go.
https://github.com/llvm/llvm-project/pull/84014
___
cfe-commits mailing list
rjmccall wrote:
> > Hmm. Is there some sort of optimization in IRGen that we need to suppress
> > here, or is it something in LLVM code gen? Presumably normal LLVM
> > optimization passes all just skip `optnone` functions.
>
> The issue #62098 demonstrates such case.
Okay. So if I
martinboehme wrote:
Forgot to push a fixup commit -- now added.
https://github.com/llvm/llvm-project/pull/84138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/martinboehme updated
https://github.com/llvm/llvm-project/pull/84138
>From 209aac483514f6d041486f36e6dabec78598fcec Mon Sep 17 00:00:00 2001
From: Martin Braenne
Date: Tue, 12 Mar 2024 15:56:41 +
Subject: [PATCH 1/2] [clang][dataflow] Make optional checker work for types
@@ -7639,6 +7639,8 @@ def err_attribute_arm_mve_polymorphism : Error<
"'__clang_arm_mve_strict_polymorphism' attribute can only be applied to an
MVE/NEON vector type">;
def err_attribute_webassembly_funcref : Error<
"'__funcref' attribute can only be applied to a function
@@ -0,0 +1,27 @@
+// RUN: %clang_cc1 -std=c2x -verify %s
+
+/* WG14 N3006: Full
+ * Underspecified object declarations
+ */
+
+struct S1 { int x, y; };// expected-note {{previous definition is
here}}
+union U1 { int a; double b; }; // expected-note {{previous definition
@@ -7813,6 +7813,32 @@ Sema::BuildCompoundLiteralExpr(SourceLocation LParenLoc,
TypeSourceInfo *TInfo,
diag::err_variable_object_no_init))
return ExprError();
}
+ } else if (LangOpts.C23 &&
AaronBallman
https://github.com/AaronBallman commented:
Thank you for working on this! I think there's something here worth
considering: how an implementation handles use of underspecified declarations
is left implementation-defined. I think we want to diagnose many of the
situations it covers, but I
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/79845
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sjanel wrote:
The [amc](https://github.com/AmadeusITGroup/amc) library defining drop-in
replacement for vector-like containers handling trivially relocatability
optimizations would love to see this kind of improvement, as currently the
users need to manually tell the compiler which types are
https://github.com/lamb-j updated
https://github.com/llvm/llvm-project/pull/85672
>From aff1a762a73ce30cde38a6fcbbed8a3e4f0b5366 Mon Sep 17 00:00:00 2001
From: Jacob Lambert
Date: Mon, 18 Mar 2024 10:19:38 -0700
Subject: [PATCH 1/4] [clang][CodeGen] Omit pre-opt link when post-opt link
https://github.com/lamb-j updated
https://github.com/llvm/llvm-project/pull/85672
>From aff1a762a73ce30cde38a6fcbbed8a3e4f0b5366 Mon Sep 17 00:00:00 2001
From: Jacob Lambert
Date: Mon, 18 Mar 2024 10:19:38 -0700
Subject: [PATCH 1/3] [clang][CodeGen] Omit pre-opt link when post-opt link
https://github.com/ilovepi closed
https://github.com/llvm/llvm-project/pull/85677
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Paul Kirth
Date: 2024-03-18T11:29:42-07:00
New Revision: 457f762651e331341872abccbb7d7724c89cbf50
URL:
https://github.com/llvm/llvm-project/commit/457f762651e331341872abccbb7d7724c89cbf50
DIFF:
https://github.com/llvm/llvm-project/commit/457f762651e331341872abccbb7d7724c89cbf50.diff
ahatanak wrote:
Any other comments? Do the changes look good?
https://github.com/llvm/llvm-project/pull/67454
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/avillega approved this pull request.
https://github.com/llvm/llvm-project/pull/85677
___
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: Paul Kirth (ilovepi)
Changes
We're seeing an issue on Macs, which shouldn't be using this config, so
we will temporarily disable this while we investigate.
---
Full diff: https://github.com/llvm/llvm-project/pull/85677.diff
1 Files
https://github.com/ilovepi created
https://github.com/llvm/llvm-project/pull/85677
We're seeing an issue on Macs, which shouldn't be using this config, so
we will temporarily disable this while we investigate.
>From 56e6059431398d126420b3595d58c3c19e20613b Mon Sep 17 00:00:00 2001
From: Paul
spavloff wrote:
> Hmm. Is there some sort of optimization in IRGen that we need to suppress
> here, or is it something in LLVM code gen? Presumably normal LLVM
> optimization passes all just skip `optnone` functions.
The issue https://github.com/llvm/llvm-project/issues/62098 demonstrates
https://github.com/scchan approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/85675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
smithp35 wrote:
> > If possible I would prefer to keep -m[no-]unaligned-access for AArch64.
> > The history of this option name derives from Arm's proprietary compiler
> >
d0k wrote:
This is now breaking multithreaded IDE use cases, can you explain why this
change is necessary?
https://github.com/llvm/llvm-project/pull/84599
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
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 705788c84623b4f1dab72a108e039a0de2d53cf6
6d6b362fbf965706108362a55c894588e80ad778 --
@@ -16570,32 +16570,53 @@ Value *CodeGenFunction::EmitPPCBuiltinExpr(unsigned
BuiltinID,
#include "llvm/TargetParser/PPCTargetParser.def"
auto GenAIXPPCBuiltinCpuExpr = [&](unsigned SupportMethod, unsigned FieldIdx,
- unsigned CompOp,
+
https://github.com/jhuber6 approved this pull request.
https://github.com/llvm/llvm-project/pull/85675
___
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-driver
Author: Yaxun (Sam) Liu (yxsamliu)
Changes
since it will cause duplicate symbols when the partially linked object is
linked again.
Change-Id: I2aea39ad0d57d3dc80b6aff395d9506ab9ebbf4d
---
Full diff:
https://github.com/yxsamliu created
https://github.com/llvm/llvm-project/pull/85675
since it will cause duplicate symbols when the partially linked object is
linked again.
Change-Id: I2aea39ad0d57d3dc80b6aff395d9506ab9ebbf4d
>From 6d6b362fbf965706108362a55c894588e80ad778 Mon Sep 17 00:00:00
https://github.com/sbc100 approved this pull request.
https://github.com/llvm/llvm-project/pull/84569
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
Thanks. Fixing it on the Chromium feels nicer to me as well. This should work
around the code if you run into more code needing this :) Personally I suspect
not, since module-level inline asm with instructions are very rarely used
(non-instruction directives are uses more).
b-sumner wrote:
This seems like it could have a significant hit on performance. Have the
runtime performance effects been measured?
https://github.com/llvm/llvm-project/pull/85672
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/lamb-j updated
https://github.com/llvm/llvm-project/pull/85672
>From aff1a762a73ce30cde38a6fcbbed8a3e4f0b5366 Mon Sep 17 00:00:00 2001
From: Jacob Lambert
Date: Mon, 18 Mar 2024 10:19:38 -0700
Subject: [PATCH 1/2] [clang][CodeGen] Omit pre-opt link when post-opt link
https://github.com/diggerlin updated
https://github.com/llvm/llvm-project/pull/82809
>From 557e7163d744890aadfa703a81a0c4f2cd112517 Mon Sep 17 00:00:00 2001
From: zhijian
Date: Fri, 23 Feb 2024 13:23:18 -0500
Subject: [PATCH 1/5] Implement a subset of builtin_cpu_supports() features
---
@@ -141,46 +149,98 @@ PPC_LNX_CPU("power10",47)
#define AIX_BUILTIN_PPC_TRUE 1
#define AIX_BUILTIN_PPC_FALSE 0
#define USE_SYS_CONF 2
-
- // Supported COMPARE_OP values.
- #define COMP_EQ 0
-
+ #define SYS_CALL 3
#endif
// The value of SUPPORT_METHOD can be
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/85501
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zmodem commented:
https://g-issues.chromium.org/issues/330163686#comment5 suggested maybe the fix
should be on the Chromium side
https://github.com/llvm/llvm-project/pull/85668
___
cfe-commits mailing list
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: Jacob Lambert (lamb-j)
Changes
Currently, when the -relink-builtin-bitcodes-postop option is used we link
builtin bitcodes twice: once before optimization, and again after optimization.
With this change, we omit the pre-opt
https://github.com/lamb-j created
https://github.com/llvm/llvm-project/pull/85672
Currently, when the -relink-builtin-bitcodes-postop option is used we link
builtin bitcodes twice: once before optimization, and again after optimization.
With this change, we omit the pre-opt linking when the
https://github.com/rjmccall approved this pull request.
Yeah, it's fine with me.
https://github.com/llvm/llvm-project/pull/85347
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
> If possible I would prefer to keep -m[no-]unaligned-access for AArch64.
>
> The history of this option name derives from Arm's proprietary compiler
>
https://github.com/AtariDreams reopened
https://github.com/llvm/llvm-project/pull/81367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AtariDreams closed
https://github.com/llvm/llvm-project/pull/81367
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/85668
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
> If Apple platforms need this option, I can keep it recognized but only for
> Apple.
I don't see any projects that use `-m(no-)unaligned-access`, but it could be
that I don't have permissions on ones that do. I'll give our qualification team
a heads-up if/when you land it,
alexcrichton wrote:
ping @sbc100, happy to answer any more questions if you have them!
I was tentatively hoping this could get backported to an 18.1.x release so we
could get a wasi-sdk release with p1/p2/etc
https://github.com/llvm/llvm-project/pull/84569
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: Fangrui Song (MaskRay)
Changes
clang-cl is conflating input assembly syntax with output assembly
syntax. It expects ATT syntax input but Intel syntax output. This
conflicts with clang -c -masm=intel users that do expect -masm=intel
https://github.com/MaskRay created
https://github.com/llvm/llvm-project/pull/85668
clang-cl is conflating input assembly syntax with output assembly
syntax. It expects AT syntax input but Intel syntax output. This
conflicts with clang -c -masm=intel users that do expect -masm=intel to
control
rjmccall wrote:
Yeah, we don't need to care about the actual bit offset of the zero-width
bit-field as long as we honor the non-interference properties across it. I'll
take a look at the patch, thanks.
https://github.com/llvm/llvm-project/pull/65742
https://github.com/spavloff updated
https://github.com/llvm/llvm-project/pull/85605
>From 5049e0209e240f0f8a3ccb6e248d55d1480b7bad Mon Sep 17 00:00:00 2001
From: Serge Pavlov
Date: Mon, 18 Mar 2024 13:20:15 +0700
Subject: [PATCH] [clang] Set correct FPOptions if attribute 'optnone' presents
Author: Joseph Huber
Date: 2024-03-18T12:12:17-05:00
New Revision: 280c7a9526a9ae7f959117c9cec94f8c8887f15c
URL:
https://github.com/llvm/llvm-project/commit/280c7a9526a9ae7f959117c9cec94f8c8887f15c
DIFF:
https://github.com/llvm/llvm-project/commit/280c7a9526a9ae7f959117c9cec94f8c8887f15c.diff
urnathan wrote:
@rjmccall here is a rebase an update, which I think addresses all your
comments. I did make some material changes though:
1) I removed the Volatile handling. I was always a little uncomfortable with it
because it didn't affect the access units of a non-volatile bitfield that
llvmbot wrote:
@llvm/pr-subscribers-llvm-ir
@llvm/pr-subscribers-clang
Author: John Brawn (john-brawn-arm)
Changes
Currently we use DW_OP_plus_uconst to handle the bitfield offset and handle the
bitfield size by choosing a type size that matches, but this doesn't work if
either offset
https://github.com/john-brawn-arm created
https://github.com/llvm/llvm-project/pull/85665
Currently we use DW_OP_plus_uconst to handle the bitfield offset and handle the
bitfield size by choosing a type size that matches, but this doesn't work if
either offset or size aren't byte-aligned.
https://github.com/gamesh411 closed
https://github.com/llvm/llvm-project/pull/80029
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Endre Fülöp
Date: 2024-03-18T17:56:15+01:00
New Revision: 705788c84623b4f1dab72a108e039a0de2d53cf6
URL:
https://github.com/llvm/llvm-project/commit/705788c84623b4f1dab72a108e039a0de2d53cf6
DIFF:
https://github.com/llvm/llvm-project/commit/705788c84623b4f1dab72a108e039a0de2d53cf6.diff
@@ -11450,6 +11450,11 @@ TEST_F(FormatTest, UnderstandsNewAndDelete) {
"void new (link p);\n"
"void delete (link p);");
+ verifyFormat("{ p->delete(); }\n"
+ "{ p->new(); }",
+ "{ p->delete (); }\n"
+ "{
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-llvm-ir
Author: Farzon Lotfi (farzonl)
Changes
this implements part 1 of 2 for #83626
- `CGBuiltin.cpp` - modified to have seperate cases for signed and unsigned
integers.
- `SemaChecking.cpp` - modified to prevent
https://github.com/farzonl created
https://github.com/llvm/llvm-project/pull/85662
this implements part 1 of 2 for #83626
- `CGBuiltin.cpp` - modified to have seperate cases for signed and unsigned
integers.
- `SemaChecking.cpp` - modified to prevent the generation of a double dot
product
@@ -54,6 +54,12 @@ ABI Changes in This Version
inline member function that contains a static local variable with a dynamic
initializer is declared with ``__declspec(dllimport)``. (#GH83616).
+- Fixed Microsoft name mangling of lifetime extended temporary objects. This
+
pranavk wrote:
I have same opinion as @lntue and Nick.
https://github.com/llvm/llvm-project/pull/85070
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
I'm happy enough with the changes so long as Swift folks are okay with the
change in behavior for existing code. CC @compnerd @rjmccall for opinions (I'll
leave the approval to one of them).
https://github.com/llvm/llvm-project/pull/85347
https://github.com/python3kgae updated
https://github.com/llvm/llvm-project/pull/83933
>From b134854e7e183a1113ee6ae5c5f7b7910270c987 Mon Sep 17 00:00:00 2001
From: Xiang Li
Date: Mon, 4 Mar 2024 16:39:41 -0800
Subject: [PATCH 01/11] [DOC][HLSL] Add documentation for root signature
This patch
alexey-bataev wrote:
You can try to run preprocessor and then manually reduce the expanded code
after preprocessor
https://github.com/llvm/llvm-project/pull/82497
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
TIFitis wrote:
Ping for review.
https://github.com/llvm/llvm-project/pull/80343
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay wrote:
"Squash and merge" uses the description instead of the commit message, so the
commit message is not useful. The commit message is due to spr.
https://github.com/llvm/llvm-project/pull/85501
___
cfe-commits mailing list
TIFitis wrote:
@alexey-bataev I'm facing issues adding the test to Clang tests as it uses the
complex.h header files. From what I can tell header files don't seem to be
supported. Is there any way to add the test you provided to the Clang tests or
a different one which can still check for the
https://github.com/Xazax-hun approved this pull request.
Thanks! At this point all of my comments are addressed, and I am happy with the
patch.
https://github.com/llvm/llvm-project/pull/85064
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -212,8 +212,23 @@ TEST(TransferTest, CopyRecordFromDerivedToBase) {
// [[p]]
}
)";
+ auto SyntheticFieldCallback = [](QualType Ty) -> llvm::StringMap {
+CXXRecordDecl *ADecl = nullptr;
+if (Ty.getAsString() == "A")
+ ADecl =
Author: Congcong Cai
Date: 2024-03-18T23:56:36+08:00
New Revision: 12b802ac0bc6ddf0742aa3fe8caecd8204d70ca5
URL:
https://github.com/llvm/llvm-project/commit/12b802ac0bc6ddf0742aa3fe8caecd8204d70ca5
DIFF:
https://github.com/llvm/llvm-project/commit/12b802ac0bc6ddf0742aa3fe8caecd8204d70ca5.diff
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/84922
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
101 - 200 of 395 matches
Mail list logo