https://github.com/arsenm approved this pull request.
https://github.com/llvm/llvm-project/pull/72709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/72107
___
cfe-commits mailing list
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
steakhal wrote:
FYI I edited the PR summary so that I'm not tagged there directly because if
someone is tagged in a commit message on GH, that person will be notified
Author: Ben Shi
Date: 2023-11-28T20:14:51+08:00
New Revision: 95a47bca5ed87bab975d9e5c841e346feee59be6
URL:
https://github.com/llvm/llvm-project/commit/95a47bca5ed87bab975d9e5c841e346feee59be6
DIFF:
https://github.com/llvm/llvm-project/commit/95a47bca5ed87bab975d9e5c841e346feee59be6.diff
LOG:
https://github.com/benshi001 closed
https://github.com/llvm/llvm-project/pull/73335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Daniel Grumberg
Date: 2023-11-28T12:55:27Z
New Revision: 6b89fab8978f2b075cecc98b3d37769b24019034
URL:
https://github.com/llvm/llvm-project/commit/6b89fab8978f2b075cecc98b3d37769b24019034
DIFF:
https://github.com/llvm/llvm-project/commit/6b89fab8978f2b075cecc98b3d37769b24019034.diff
https://github.com/daniel-grumberg closed
https://github.com/llvm/llvm-project/pull/73369
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/usx95 updated https://github.com/llvm/llvm-project/pull/72348
>From b29aa485f2a541243d3764f4ed711ccc5a869519 Mon Sep 17 00:00:00 2001
From: Utkarsh Saxena
Date: Wed, 15 Nov 2023 06:26:19 +0100
Subject: [PATCH 1/4] [STLExctras] Add out-of-line definition of friend
operator==
@@ -2282,6 +2282,15 @@ Instruction
*InstCombinerImpl::visitGetElementPtrInst(GetElementPtrInst ) {
if (MadeChange)
return
+ // Canonicalize constant GEPs to i8 type.
bjope wrote:
Right. So things can be expected to just work (given
https://github.com/tbaederr approved this pull request.
https://github.com/llvm/llvm-project/pull/73644
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Michael137 created
https://github.com/llvm/llvm-project/pull/73626
In #71780 we started emitting definitions for all static data-members with
constant initialisers, even if they were constants (i.e., didn't have a
location). We also dropped the DW_AT_const_value from the
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-debuginfo
Author: Michael Buch (Michael137)
Changes
In #71780 we started emitting definitions for all static data-members
with constant initialisers, even if they were constants (i.e.,
Michael137 wrote:
FYI @petrhosek
https://github.com/llvm/llvm-project/pull/73626
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Michael137 edited
https://github.com/llvm/llvm-project/pull/73626
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MDevereau updated
https://github.com/llvm/llvm-project/pull/73304
>From bca5297fe897edf6faf51ffde3e8fe1baa26b148 Mon Sep 17 00:00:00 2001
From: Matt Devereau
Date: Mon, 20 Nov 2023 15:50:28 +
Subject: [PATCH 1/3] [SME2] Add LUTI2 and LUTI4 single Builtins and Intrinsics
https://github.com/benshi001 updated
https://github.com/llvm/llvm-project/pull/73335
>From b520a4aee77f36622e82b12a32fee54ed20b07d0 Mon Sep 17 00:00:00 2001
From: Ben Shi
Date: Fri, 24 Nov 2023 22:51:27 +0800
Subject: [PATCH] [clang][analyzer] Support `fputs` in the StreamChecker
---
benshi001 wrote:
The CI test seems being pending, so I do a new `push` to refresh it.
https://github.com/llvm/llvm-project/pull/73335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vikramRH wrote:
ping
https://github.com/llvm/llvm-project/pull/72556
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
steakhal wrote:
> @steakhal thanks for the checking and sorry for the unintentional spamming.
>
> > Such a great feature, right?
>
> Just wonderful
To clarify, you
@@ -2282,6 +2282,15 @@ Instruction
*InstCombinerImpl::visitGetElementPtrInst(GetElementPtrInst ) {
if (MadeChange)
return
+ // Canonicalize constant GEPs to i8 type.
bjope wrote:
FWIW, this is a bit interesting downstream with non-8-bit addressing
arsenm wrote:
ping
https://github.com/llvm/llvm-project/pull/68267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Zhikai Zeng
Date: 2023-11-28T22:08:11+08:00
New Revision: 01091fd101ee78fb8068a2b916ae007a4873d66a
URL:
https://github.com/llvm/llvm-project/commit/01091fd101ee78fb8068a2b916ae007a4873d66a
DIFF:
https://github.com/llvm/llvm-project/commit/01091fd101ee78fb8068a2b916ae007a4873d66a.diff
@@ -6334,8 +6334,8 @@ defm loop_versioning : BoolOptionWithoutMarshalling<"f",
"version-loops-for-stri
PosFlag,
NegFlag>;
defm alias_analysis : BoolOptionWithoutMarshalling<"f", "alias-analysis",
- PosFlag,
- NegFlag>;
+ PosFlag,
+ NegFlag>;
} // let Visibility =
https://github.com/benshi001 updated
https://github.com/llvm/llvm-project/pull/73335
>From ff1ab48c5760302dc55ff889b5b3dfd0b884aa33 Mon Sep 17 00:00:00 2001
From: Ben Shi
Date: Fri, 24 Nov 2023 22:51:27 +0800
Subject: [PATCH 1/2] [clang][analyzer] Support `fputs` in the StreamChecker
---
sylvestre wrote:
Merci à toi :)
https://github.com/llvm/llvm-project/pull/73234
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Michael137 wrote:
I do wonder how feasible it would be for the downstream tests to be adjusted to
look at the `DW_AT_location`..
https://github.com/llvm/llvm-project/pull/73626
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
Author: serge-sans-paille
Date: 2023-11-28T13:46:37Z
New Revision: ced0f28a35b8a14759e6bd9418e8c8a7c35775c8
URL:
https://github.com/llvm/llvm-project/commit/ced0f28a35b8a14759e6bd9418e8c8a7c35775c8
DIFF:
https://github.com/llvm/llvm-project/commit/ced0f28a35b8a14759e6bd9418e8c8a7c35775c8.diff
https://github.com/serge-sans-paille closed
https://github.com/llvm/llvm-project/pull/73491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/serge-sans-paille updated
https://github.com/llvm/llvm-project/pull/73491
>From c66e50ee0c804eadd8cb09650f41cc31a76e8bdd Mon Sep 17 00:00:00 2001
From: serge-sans-paille
Date: Mon, 27 Nov 2023 10:17:32 +0100
Subject: [PATCH 1/2] Improve clang-format-diff help output
It is
sebastianpoeplau wrote:
I just discovered #28205 which should be fixed by this change.
https://github.com/llvm/llvm-project/pull/72400
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -560,14 +560,15 @@ define i32 @test28() nounwind {
; CHECK-NEXT: entry:
; CHECK-NEXT:[[ORIENTATIONS:%.*]] = alloca [1 x [1 x %struct.x]], align 8
; CHECK-NEXT:[[T3:%.*]] = call i32 @puts(ptr noundef nonnull
dereferenceable(1) @.str) #[[ATTR0]]
+; CHECK-NEXT:
serge-sans-paille wrote:
> See below. Seems it's already working as expected without this patch.
>
> ```
> $ clang-format -version
> clang-format version 17.0.5
> $ clang-format-diff.py foo
> usage: clang-format-diff.py [-h] [-i] [-p NUM] [-regex PATTERN] [-iregex
> PATTERN]
>
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/73493
>From d3cb9b147c443762fc7d545100144e462bbe3b58 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Mon, 27 Nov 2023 10:48:13 +0100
Subject: [PATCH 1/2] [Clang] CWG2789 Overload resolution with implicit and
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy ,
=?utf-8?q?Donát?= Nagy
Message-ID:
In-Reply-To:
@@ -350,17 +383,38 @@ void ArrayBoundCheckerV2::checkLocation(SVal Location,
bool IsLoad,
if (ExceedsUpperBound) {
if (!WithinUpperBound) {
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/73491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/benshi001 created
https://github.com/llvm/llvm-project/pull/73638
This PR contains two commits, the first one is identical to
https://github.com/llvm/llvm-project/pull/73335, please just review the second
one about `fgets`.
>From 31d7b6c5ddbbe37d59819fd7728e817102a9d24b
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Ben Shi (benshi001)
Changes
This PR contains two commits, the first one is identical to
https://github.com/llvm/llvm-project/pull/73335, please just review the second
one about `fgets`.
---
Full diff:
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Zhikai Zeng (Backl1ght)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/73644.diff
1 Files Affected:
- (modified) clang/lib/Sema/SemaInit.cpp (+1-1)
``diff
diff --git a/clang/lib/Sema/SemaInit.cpp
@@ -104,3 +106,17 @@ void fun() {
(void) b;
(void) var_host_only;
}
+
+// NEG-NOT: external_func
+extern __global__ void external_func();
+// NEG-NOT: @external_dep
+extern void* const external_dep[] = {
+ (void*)(external_func)
+};
+// NEG-NOT: @external_arr
https://github.com/cor3ntin approved this pull request.
Lgtm
https://github.com/llvm/llvm-project/pull/73644
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Backl1ght updated
https://github.com/llvm/llvm-project/pull/73644
>From a417fb4d421cc28115bb2ea2062fb400586a7042 Mon Sep 17 00:00:00 2001
From: Backl1ght
Date: Tue, 28 Nov 2023 21:18:49 +0800
Subject: [PATCH] [clang] fix typo
---
clang/lib/Sema/SemaInit.cpp | 2 +-
1 file
sylvestre wrote:
Seems it caused:
https://github.com/llvm/llvm-project/issues/73628
https://github.com/llvm/llvm-project/pull/73234
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
Please open a github issue to show us what you are trying to accomplish and
reference the issue number here.
https://github.com/llvm/llvm-project/pull/73491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/serge-sans-paille updated
https://github.com/llvm/llvm-project/pull/73491
>From c66e50ee0c804eadd8cb09650f41cc31a76e8bdd Mon Sep 17 00:00:00 2001
From: serge-sans-paille
Date: Mon, 27 Nov 2023 10:17:32 +0100
Subject: [PATCH 1/2] Improve clang-format-diff help output
It is
Author: Corentin Jabot
Date: 2023-11-28T12:05:34+01:00
New Revision: 4142a64ae3ca4856e6a5ffae9e40014ef5cf9db5
URL:
https://github.com/llvm/llvm-project/commit/4142a64ae3ca4856e6a5ffae9e40014ef5cf9db5
DIFF:
https://github.com/Benjins updated
https://github.com/llvm/llvm-project/pull/73592
>From de348ecdbf9d3c299eb4fe302ed2c224df7cde6b Mon Sep 17 00:00:00 2001
From: Benji Smith <6193112+benj...@users.noreply.github.com>
Date: Mon, 27 Nov 2023 18:15:22 -0500
Subject: [PATCH] [C API] Add support for
Author: cor3ntin
Date: 2023-11-28T13:30:18+01:00
New Revision: 205f53010d0da11a9c4ab7a4655edb4a2ded1b02
URL:
https://github.com/llvm/llvm-project/commit/205f53010d0da11a9c4ab7a4655edb4a2ded1b02
DIFF:
https://github.com/llvm/llvm-project/commit/205f53010d0da11a9c4ab7a4655edb4a2ded1b02.diff
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/73493
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
spavloff wrote:
LGTM.
https://github.com/llvm/llvm-project/pull/67302
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yangxili2023 wrote:
Thank you everybody! The problem is solved.
https://github.com/llvm/llvm-project/pull/73096
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
benshi001 wrote:
> I prefer to have the new test code with
> `StreamTesterChecker_make_feof_stream` in a new test function (with
> appropriate name like `write_after_eof_is_allowed`), to improve code
> maintainability (`fputc` and `fputs` can be in the same function).
>
> In github PR's
cor3ntin wrote:
@sylvestre Thanks. It seems like GCC 7.5 does not select the appropriate
constructor https://compiler-explorer.com/z/oc315aYnT - I will commit a fix in
a bit.
https://github.com/llvm/llvm-project/pull/73234
___
cfe-commits mailing
serge-sans-paille wrote:
@owenca bug opened as #73635 and commit message updated to link to it.
https://github.com/llvm/llvm-project/pull/73491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
cor3ntin wrote:
> I _think_ the new map should be unnecessary: `FunctionTemplateDecl` and
> `VarTemplateDecl` already have a list of specializations that you can walk
> and check the point of instantiation and template specialization kind to see
> if there's a pending instantiation.
I don't
https://github.com/benshi001 updated
https://github.com/llvm/llvm-project/pull/73638
>From c6cfabce282dc0a493c1d057c0c83274a1c51634 Mon Sep 17 00:00:00 2001
From: Ben Shi
Date: Tue, 28 Nov 2023 19:27:21 +0800
Subject: [PATCH] [clang][analyzer] Support `fgets` in the StreamChecker
---
https://github.com/benshi001 edited
https://github.com/llvm/llvm-project/pull/73638
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DominikAdamski updated
https://github.com/llvm/llvm-project/pull/72638
>From 1cc1d9d3480f750980ea0a395cca3b202a606f57 Mon Sep 17 00:00:00 2001
From: Dominik Adamski
Date: Fri, 17 Nov 2023 03:02:49 -0600
Subject: [PATCH 1/2] [Flang] Add code-object-version option
Information
Fznamznon wrote:
Ping.
https://github.com/llvm/llvm-project/pull/73099
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2282,6 +2282,15 @@ Instruction
*InstCombinerImpl::visitGetElementPtrInst(GetElementPtrInst ) {
if (MadeChange)
return
+ // Canonicalize constant GEPs to i8 type.
nikic wrote:
Note that i8 GEPs are already generated by some important components
https://github.com/Backl1ght closed
https://github.com/llvm/llvm-project/pull/73644
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jryans approved this pull request.
Thanks, seems like a fine temporary measure to me!
https://github.com/llvm/llvm-project/pull/73626
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/balazske approved this pull request.
https://github.com/llvm/llvm-project/pull/73335
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
> ping
The split up parts are still part of this one PR. Currently you're supposed to
create a separate PR for each separate change. The set behavior is to squash
all of these together on submit
https://github.com/llvm/llvm-project/pull/72556
AaronBallman wrote:
> I guess I'd consider the "mental model" here to be that (notionally) `#embed`
> is preprocessed by expanding to `#embed_base64`, which is handled by the
> compiler proper, not the preprocessor. Yes, that's not entirely true in the
> implementation, but it seems like a
@@ -6334,8 +6334,8 @@ defm loop_versioning : BoolOptionWithoutMarshalling<"f",
"version-loops-for-stri
PosFlag,
NegFlag>;
defm alias_analysis : BoolOptionWithoutMarshalling<"f", "alias-analysis",
- PosFlag,
- NegFlag>;
+ PosFlag,
+ NegFlag>;
} // let Visibility =
https://github.com/banach-space edited
https://github.com/llvm/llvm-project/pull/73548
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/banach-space approved this pull request.
Thanks, LGTM!
> @banach-space I'm happy to change the behavior too if you'd prefer that
That would be great, but could be done independently. Up to you!
In general, I feel that `-f{no-}alias-analysis` interacts with `-O{#N}` in a
robozati wrote:
@HighCommander4, thank you for reviewing this PR!
Unfortunately, I’m currently in my finals and will only have time to look at
this next week. So, I’ll update this later.
https://github.com/llvm/llvm-project/pull/69849
___
=?utf-8?q?Don=C3=A1t?= Nagy ,
=?utf-8?q?Don=C3=A1t?= Nagy ,
=?utf-8?q?Don=C3=A1t?= Nagy ,
=?utf-8?q?Don=C3=A1t?= Nagy
Message-ID:
In-Reply-To:
DonatNagyE wrote:
@steakhal thanks for the checking and sorry for the unintentional spamming.
> Such a great feature, right?
Just wonderful :smile:
nicovank wrote:
Ping. @carlosgalvezp could you please weigh in? modernize / performance? Anyone
else?
I have no real preference either way. Now that modernize is fully integrated in
Clang-Tidy, I don't think historical reasons should lead to modernize
overbloat. If this check was only
@@ -2282,6 +2282,15 @@ Instruction
*InstCombinerImpl::visitGetElementPtrInst(GetElementPtrInst ) {
if (MadeChange)
return
+ // Canonicalize constant GEPs to i8 type.
nikic wrote:
GEP operates in terms of bytes, not bits. The size of i8 is required
https://github.com/Backl1ght created
https://github.com/llvm/llvm-project/pull/73644
None
>From a417fb4d421cc28115bb2ea2062fb400586a7042 Mon Sep 17 00:00:00 2001
From: Backl1ght
Date: Tue, 28 Nov 2023 21:18:49 +0800
Subject: [PATCH] [clang] fix typo
---
clang/lib/Sema/SemaInit.cpp | 2 +-
1
https://github.com/Backl1ght updated
https://github.com/llvm/llvm-project/pull/73644
>From 1381fd047b27a019166b6c9552c55afaa916ee39 Mon Sep 17 00:00:00 2001
From: Backl1ght
Date: Tue, 28 Nov 2023 21:18:49 +0800
Subject: [PATCH] [clang] fix typo
---
clang/lib/Sema/SemaInit.cpp | 2 +-
1 file
AMP999 wrote:
@cor3ntin Could you land this for me, please?
https://github.com/llvm/llvm-project/pull/69734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yxsamliu approved this pull request.
LGTM. Thanks
https://github.com/llvm/llvm-project/pull/68267
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
@AMP999 can you resolve the conflict?
https://github.com/llvm/llvm-project/pull/69734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AlexVlx updated
https://github.com/llvm/llvm-project/pull/69266
>From ded7435220d2c3527c4798d1b328a5f2940e279a Mon Sep 17 00:00:00 2001
From: Alex Voicu
Date: Mon, 16 Oct 2023 22:43:55 +0100
Subject: [PATCH 1/3] Handle trying to bind a generic reference to a template
https://github.com/tomekpaszek updated
https://github.com/llvm/llvm-project/pull/70338
>From b5ba0b3fde2c6662e19dfdf96a787621ec767460 Mon Sep 17 00:00:00 2001
From: Tomek Paszek
Date: Sat, 11 Nov 2023 19:38:00 +0100
Subject: [PATCH 01/15] Added an option to ignore macro definitions.
---
https://github.com/nikic created https://github.com/llvm/llvm-project/pull/73662
This adds support for using dominating conditions in computeKnownBits() when
called from InstCombine. The implementation uses a DomConditionCache, which
stores which branches may provide information that is
@@ -977,14 +977,61 @@ bool tools::addOpenMPRuntime(ArgStringList ,
const ToolChain ,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain ,
+void tools::addFortranRuntimeLibs(const ToolChain , const ArgList ,
https://github.com/mjklemm updated
https://github.com/llvm/llvm-project/pull/73124
>From 2a2693364cb8e9b657b9ff54aa78df0466b55fe4 Mon Sep 17 00:00:00 2001
From: Michael Klemm
Date: Wed, 22 Nov 2023 14:22:20 +0100
Subject: [PATCH 01/16] Let the linker fail on multiple definitions of main()
---
@@ -1861,6 +1861,13 @@ bool CompilerInvocation::ParseCodeGenArgs(CodeGenOptions
, ArgList ,
if (Args.hasArg(OPT_funified_lto))
Opts.PrepareForThinLTO = true;
}
+ if (Arg *A = Args.getLastArg(options::OPT_ffat_lto_objects,
+
ronlieb wrote:
@jdoerfert please respond to our comments about peformance degradation ?
https://github.com/llvm/llvm-project/pull/70401
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/tomekpaszek updated
https://github.com/llvm/llvm-project/pull/70338
>From b5ba0b3fde2c6662e19dfdf96a787621ec767460 Mon Sep 17 00:00:00 2001
From: Tomek Paszek
Date: Sat, 11 Nov 2023 19:38:00 +0100
Subject: [PATCH 01/15] Added an option to ignore macro definitions.
---
https://github.com/silee2 updated
https://github.com/llvm/llvm-project/pull/71430
>From c76403cf8629b8f7d8a5b7a3ee5da2881713a7f8 Mon Sep 17 00:00:00 2001
From: "Lee, Sang Ik"
Date: Mon, 6 Nov 2023 18:47:23 +
Subject: [PATCH 1/5] [MLIR] Enable GPU Dialect to SYCL runtime integration
GPU
ZijunZhaoCCK wrote:
> Gentle ping. There's outstanding feedback to address on this review.
> @ZijunZhaoCCK if you don't think you'll have time to pursue this PR anymore,
> it's all good but please let us know so we can assign it to someone else!
Sorry for the late reply! I can’t make it a
@@ -29,30 +29,31 @@ Overview
Within LLVM, FatLTO is supported by choosing the ``FatLTODefaultPipeline``.
This pipeline will:
-#) Clone the IR module.
-#) Run the pre-link (Thin)LTO pipeline using the cloned module.
+#) Run the pre-link UnifiedLTO pipeline on the current
@@ -9,6 +9,17 @@
// Defines the API between compiled code and the implementations of
time-related
// intrinsic subroutines in the runtime library.
+// time-intrinsic.h
+#ifndef TIME_INTRINSIC_H
+#define TIME_INTRINSIC_H
+
+#include
+
+void copyBufferAndPad(
+char *dest,
wangpc-pp wrote:
> short version: GCC isn't change. long version: GCC's configure script isn't
> change, it's configure script in riscv-gnu-toolchain
So why is there a difference between GCC and riscv-gnu-toolchain? If we set
`with_abi` to lp64f, what is the behavior?
>
> But I don't have
@@ -0,0 +1,31 @@
+//===- SPIRVToLLVMIRTranslation.cpp - Translate SPIRV to LLVM IR
--===//
+//
+// 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:
cor3ntin wrote:
> So it's not so much that it's not actually true in the implementation
> details, it's that it's not actually true by specification either.
+1. I really do not want `-E` to start not expanding, or transforming
directives.
I do not see a difference between a large embed and a
https://github.com/mjklemm updated
https://github.com/llvm/llvm-project/pull/73124
>From 2a2693364cb8e9b657b9ff54aa78df0466b55fe4 Mon Sep 17 00:00:00 2001
From: Michael Klemm
Date: Wed, 22 Nov 2023 14:22:20 +0100
Subject: [PATCH 01/16] Let the linker fail on multiple definitions of main()
---
usx95 wrote:
Update: I verified that this change works fine with clang-17.0.6 and fails with
clang-17.0.5 in C++20.
https://github.com/llvm/llvm-project/pull/72348
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -977,14 +977,61 @@ bool tools::addOpenMPRuntime(ArgStringList ,
const ToolChain ,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain ,
+void tools::addFortranRuntimeLibs(const ToolChain , const ArgList ,
@@ -977,14 +977,61 @@ bool tools::addOpenMPRuntime(ArgStringList ,
const ToolChain ,
return true;
}
-void tools::addFortranRuntimeLibs(const ToolChain ,
+void tools::addFortranRuntimeLibs(const ToolChain , const ArgList ,
plotfi wrote:
> Pinging @nikic for another round of review.
@nikic is there more feedback or can this PR be merged?
https://github.com/llvm/llvm-project/pull/68176
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
elizabethandrews wrote:
Thank you reviews! I apologize for the delay in my response. I was OOO last
week. I will apply review comments and push an update ASAP.
https://github.com/llvm/llvm-project/pull/71706
___
cfe-commits mailing list
AMP999 wrote:
Oh, Sure. I didn't noticed that.
https://github.com/llvm/llvm-project/pull/69734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arichardson wrote:
> > The commit message is slightly misleading, it appears there are some parts
> > of the file that are actually used?
>
> As the commit message says, we only use
> https://github.com/llvm/llvm-project/blob/main/libunwind/src/Unwind-wasm.c
> and not libunwind.cpp. We don't
AMP999 wrote:
@cor3ntin I've fixed that.
https://github.com/llvm/llvm-project/pull/69734
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 366 matches
Mail list logo