https://github.com/kadircet created
https://github.com/llvm/llvm-project/pull/65444:
None
From 0b33a1277beb1ec518cb000923d02d78365bfd82 Mon Sep 17 00:00:00 2001
From: Kadir Cetinkaya
Date: Tue, 5 Sep 2023 19:19:47 +0200
Subject: [PATCH] [clangd][unittests] Limit paralelism for clangd unittests
https://github.com/kadircet review_requested
https://github.com/llvm/llvm-project/pull/65444
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet review_requested
https://github.com/llvm/llvm-project/pull/65444
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65444
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/65444
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kadircet wrote:
@HighCommander4 @AaronBallman FYI
https://github.com/llvm/llvm-project/pull/65444
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hazohelet added a comment.
@cor3ntin Gentle ping
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155610/new/
https://reviews.llvm.org/D155610
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinf
https://github.com/davemgreen review_requested
https://github.com/llvm/llvm-project/pull/65423
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin updated this revision to Diff 555973.
cor3ntin added a comment.
Rebase
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159126/new/
https://reviews.llvm.org/D159126
Files:
clang/include/clang/Sema/Sema.h
clang/lib/Sema/SemaConcept.cpp
cor3ntin updated this revision to Diff 555974.
cor3ntin added a comment.
Rebase and cleanups
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159126/new/
https://reviews.llvm.org/D159126
Files:
clang/docs/ReleaseNotes.rst
clang/include/clang/Sema
cor3ntin added a comment.
@erichkeane @aaron.ballman This now pass the tests and is ready for review :)
Thanks to @beanz for the quick turnaround on the HLSL issue
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159126/new/
https://reviews.llvm.org/
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/65431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet approved this pull request.
thanks!
https://github.com/llvm/llvm-project/pull/65431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -199,6 +200,9 @@ void IncludeCleanerCheck::check(const
MatchFinder::MatchResult &Result) {
tooling::HeaderIncludes HeaderIncludes(getCurrentMainFile(), Code,
FileStyle->IncludeStyle);
+ // `tooling::HeaderIncludes::insert` will n
@@ -224,7 +224,8 @@ Changes in existing checks
- Improved :doc:`misc-include-cleaner
` check by adding option
- `DeduplicateFindings` to output one finding per symbol occurrence.
+ `DeduplicateFindings` to output one finding per symbol occurrence
+ and avoid fixes insert
@@ -209,14 +213,17 @@ void IncludeCleanerCheck::check(const
MatchFinder::MatchResult &Result) {
// main file.
if (auto Replacement =
HeaderIncludes.insert(llvm::StringRef{Spelling}.trim("\"<>"),
- Angled, tooling::IncludeDir
https://github.com/danix800 created
https://github.com/llvm/llvm-project/pull/65448:
Fixes #54377
>From 2b04f714f1e1985546ff7ed00c131591316782b8 Mon Sep 17 00:00:00 2001
From: dingfei
Date: Wed, 6 Sep 2023 10:03:21 +0800
Subject: [PATCH] [analyzer] Reduce constraint on modulo with small concre
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet labeled
https://github.com/llvm/llvm-project/pull/65431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
With just the bitvector change this is fine.
I don't disagree with splitting the test files but it should be in its own
commit/PR.
https://github.com/llvm/llvm-project/pull/65423
___
cfe-commits mailing list
cfe-commits@lists.llv
https://github.com/danix800 updated
https://github.com/llvm/llvm-project/pull/65448:
>From 235f39a6a5137e53239105727798d4540e5dd563 Mon Sep 17 00:00:00 2001
From: dingfei
Date: Wed, 6 Sep 2023 10:03:21 +0800
Subject: [PATCH] [analyzer] Reduce constraint on modulo with small concrete
range
---
kadircet added a comment.
see also https://github.com/llvm/llvm-project/issues/62172
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159441/new/
https://reviews.llvm.org/D159441
___
cfe-commits mailing lis
Author: martinboehme
Date: 2023-09-06T09:43:05+02:00
New Revision: f470c361d959ec21dee51f65b53412ff8a63c946
URL:
https://github.com/llvm/llvm-project/commit/f470c361d959ec21dee51f65b53412ff8a63c946
DIFF:
https://github.com/llvm/llvm-project/commit/f470c361d959ec21dee51f65b53412ff8a63c946.diff
https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/65329
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
Whatever we do, I consider it important for us to understand how widely `cmd`
is used.
https://github.com/llvm/llvm-project/pull/65242
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listin
Author: martinboehme
Date: 2023-09-06T09:52:30+02:00
New Revision: be12f26dc00cad8bcd575b176a453a1c7c4fdff5
URL:
https://github.com/llvm/llvm-project/commit/be12f26dc00cad8bcd575b176a453a1c7c4fdff5
DIFF:
https://github.com/llvm/llvm-project/commit/be12f26dc00cad8bcd575b176a453a1c7c4fdff5.diff
https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/65322
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mboehme accepted this revision.
mboehme added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/unittests/Analysis/FlowSensitive/TransferTest.cpp:2498
+TEST(TransferTest, ReturnStructWithInheritance) {
+ // This is a crash repro. (Returning a st
https://github.com/martinboehme closed
https://github.com/llvm/llvm-project/pull/65301
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: martinboehme
Date: 2023-09-06T10:02:21+02:00
New Revision: c0703eaec1df4cde861f15e244507ef79f2d3d82
URL:
https://github.com/llvm/llvm-project/commit/c0703eaec1df4cde861f15e244507ef79f2d3d82
DIFF:
https://github.com/llvm/llvm-project/commit/c0703eaec1df4cde861f15e244507ef79f2d3d82.diff
@@ -121,18 +121,18 @@ static Value *mergeDistinctValues(QualType Type, Value
&Val1,
Value *MergedVal = nullptr;
if (auto *RecordVal1 = dyn_cast(&Val1)) {
-[[maybe_unused]] auto *RecordVal2 = cast(&Val2);
-
-// Values to be merged are always associated with the sam
@@ -121,18 +121,18 @@ static Value *mergeDistinctValues(QualType Type, Value
&Val1,
Value *MergedVal = nullptr;
if (auto *RecordVal1 = dyn_cast(&Val1)) {
-[[maybe_unused]] auto *RecordVal2 = cast(&Val2);
-
-// Values to be merged are always associated with the sam
nridge accepted this revision.
nridge added a subscriber: rjmccall.
nridge added a comment.
This revision is now accepted and ready to land.
I'm not well versed in the parser code, but the fix clearly avoids the
assertion in `getIdentifierInfo()`, and both @aaron.ballman and @rjmccall
indicated
https://github.com/sdesmalen-arm edited
https://github.com/llvm/llvm-project/pull/65423
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sdesmalen-arm wrote:
Would it have been possible to change ARMCPUTestParams to give it two
constructors, one taking a uint64_t mask (for the ARM features), and the other
constructor taking a BitVector (for the AArch64 features), and then to have the
classes w
@@ -260,7 +262,7 @@ inline constexpr ExtensionInfo Extensions[] = {
{"tme", AArch64::AEK_TME, "+tme", "-tme", FEAT_MAX, "", 0},
{"wfxt", AArch64::AEK_NONE, {}, {}, FEAT_WFXT, "+wfxt", 550},
{"gcs", AArch64::AEK_GCS, "+gcs", "-gcs", FEAT_MAX, "", 0},
-// Special
@@ -315,23 +317,23 @@ struct ArchInfo {
};
// clang-format off
-inline constexpr ArchInfo ARMV8A= { VersionTuple{8, 0}, AProfile,
"armv8-a", "+v8a", (AArch64::AEK_FP | AArch64::AEK_SIMD), };
-inline constexpr ArchInfo ARMV8_1A = { VersionTuple{8, 1}, AProfile,
"armv8.1-
https://github.com/sdesmalen-arm commented:
Seems like a sensible refactor, we're going to run out of bits eventually.
https://github.com/llvm/llvm-project/pull/65423
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
@@ -315,23 +317,23 @@ struct ArchInfo {
};
// clang-format off
-inline constexpr ArchInfo ARMV8A= { VersionTuple{8, 0}, AProfile,
"armv8-a", "+v8a", (AArch64::AEK_FP | AArch64::AEK_SIMD), };
-inline constexpr ArchInfo ARMV8_1A = { VersionTuple{8, 1}, AProfile,
"armv8.1-
@@ -96,64 +97,65 @@ static_assert(FEAT_MAX <= 64,
// Arch extension modifiers for CPUs. These are labelled with their Arm ARM
// feature name (though the canonical reference for those is AArch64.td)
// clang-format off
-enum ArchExtKind : uint64_t {
- AEK_NONE =1,
- A
VitaNuo updated this revision to Diff 555985.
VitaNuo added a comment.
Fix analysis test to properly recognize the resource directory.
Use HeaderSearch->getModuleMap().BuiltinDir to find out the resource directory.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://review
SuH added inline comments.
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoZicfiss.td:76
+let Uses = [SSP], Defs = [SSP], hasSideEffects = 0, mayLoad = 0, mayStore = 1
in {
+def SSPUSH : RVInstR<0b1000101, 0b100, OPC_SYSTEM, (outs), (ins GPRRA:$rs2),
+ "sspus
VitaNuo marked an inline comment as done.
VitaNuo added a comment.
Thanks for the help in resolving the resource dir issues!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157610/new/
https://reviews.llvm.org/D157610
___
jmmartinez updated this revision to Diff 555986.
jmmartinez marked an inline comment as done.
jmmartinez added a comment.
- Review
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159206/new/
https://reviews.llvm.org/D159206
Files:
clang/lib/CodeGe
https://github.com/sam-mccall approved this pull request.
https://github.com/llvm/llvm-project/pull/65444
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jmmartinez updated this revision to Diff 555987.
jmmartinez marked 2 inline comments as done.
jmmartinez added a comment.
- Capitalize comment
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D159206/new/
https://reviews.llvm.org/D159206
Files:
clan
jmmartinez added inline comments.
Comment at: clang/lib/CodeGen/CGCall.cpp:2030-2031
+ bool EnabledForTarget = TEntry->second;
+ if (EnabledForTarget != EnabledForFunc)
+return;
+}
arsenm wrote:
> jmmartinez wrote:
> > arsenm wrote:
> > > Ea
https://github.com/HerrCai0907 resolved
https://github.com/llvm/llvm-project/pull/65431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 resolved
https://github.com/llvm/llvm-project/pull/65431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 resolved
https://github.com/llvm/llvm-project/pull/65431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/65431:
>From 2b727285edb91a4a88add118745eabc08da9c6fd Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 6 Sep 2023 09:55:20 +0800
Subject: [PATCH 1/2] [clang-tidy][misc-include-cleaner]Avoid fixes insert same
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/65431:
>From 2b727285edb91a4a88add118745eabc08da9c6fd Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 6 Sep 2023 09:55:20 +0800
Subject: [PATCH 1/2] [clang-tidy][misc-include-cleaner]Avoid fixes insert same
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/65431:
>From 2b727285edb91a4a88add118745eabc08da9c6fd Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 6 Sep 2023 09:55:20 +0800
Subject: [PATCH 1/2] [clang-tidy][misc-include-cleaner]Avoid fixes insert same
Allen added inline comments.
Comment at: clang/lib/Sema/SemaDecl.cpp:11544
+ // Target attribute on AArch64 is not used for multiversioning
+ if (NewTA && S.getASTContext().getTargetInfo().getTriple().isAArch64())
+return false;
I find the attribute **targe
Author: Kadir Cetinkaya
Date: 2023-09-06T10:53:18+02:00
New Revision: 9a26d2c6d35f574d7a4b06a5a22f8a1c063cb664
URL:
https://github.com/llvm/llvm-project/commit/9a26d2c6d35f574d7a4b06a5a22f8a1c063cb664
DIFF:
https://github.com/llvm/llvm-project/commit/9a26d2c6d35f574d7a4b06a5a22f8a1c063cb664.dif
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/65246
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/65431:
>From 2b727285edb91a4a88add118745eabc08da9c6fd Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 6 Sep 2023 09:55:20 +0800
Subject: [PATCH 1/3] [clang-tidy][misc-include-cleaner]Avoid fixes insert same
uabelho added inline comments.
Comment at: clang/lib/Lex/InitHeaderSearch.cpp:336
- case llvm::Triple::Minix:
-AddGnuCPlusPlusIncludePaths("/usr/gnu/include/c++/4.4.3",
-"", "", "", triple);
@brad :
I think this was the last u
https://github.com/phoebewang created
https://github.com/llvm/llvm-project/pull/65457:
m_x86_Features_Group always turn `mno-` into `-target-feature-`. In
this case, we don't have `-gather/-scatter` but `+prefer-no-gather/scatter`.
>From be58af68f221bb65788e74f8cfe4952c1038ae70 Mon Sep
https://github.com/phoebewang review_requested
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mstorsjo wrote:
CC @cjacek @efriedma-quic
https://github.com/llvm/llvm-project/pull/65420
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/phoebewang edited
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/phoebewang edited
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/phoebewang review_requested
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Congcong Cai
Date: 2023-09-06T17:37:14+08:00
New Revision: b0831c3996752038c375796d8ebb4f471f1ea251
URL:
https://github.com/llvm/llvm-project/commit/b0831c3996752038c375796d8ebb4f471f1ea251
DIFF:
https://github.com/llvm/llvm-project/commit/b0831c3996752038c375796d8ebb4f471f1ea251.diff
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/65431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
VitaNuo updated this revision to Diff 555995.
VitaNuo added a comment.
Rebase to current main.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157610/new/
https://reviews.llvm.org/D157610
Files:
clang-tools-extra/clang-tidy/misc/IncludeCleanerChec
VitaNuo created this revision.
Herald added subscribers: kadircet, arphaman, xazax.hun.
Herald added a project: All.
VitaNuo requested review of this revision.
Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov.
Herald added a project: clang.
Repository:
rG LLVM Github Monorepo
https
https://github.com/XinWang10 approved this pull request.
LGTM, could work on my side.
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/phoebewang edited
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Phoebe Wang
Date: 2023-09-06T18:08:29+08:00
New Revision: c5fabaccef5a8566bd65a6f242be65d75096bea8
URL:
https://github.com/llvm/llvm-project/commit/c5fabaccef5a8566bd65a6f242be65d75096bea8
DIFF:
https://github.com/llvm/llvm-project/commit/c5fabaccef5a8566bd65a6f242be65d75096bea8.diff
L
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/65457
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenpan accepted this revision.
owenpan added a comment.
LG with a couple of minor comments.
Comment at: clang/lib/Format/ContinuationIndenter.cpp:334
+ // enabled.
+ !(Current.is(TT_LambdaLBrace) && Style.BraceWrapping.BeforeLambdaBody) &&
CurrentState.NoLineB
void added a comment.
This is now ready for a non-WIP review. PTAL. :-)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148381/new/
https://reviews.llvm.org/D148381
___
cfe-commits mailing list
cfe-commits
ilinpv added inline comments.
Comment at: clang/lib/Sema/SemaDecl.cpp:11544
+ // Target attribute on AArch64 is not used for multiversioning
+ if (NewTA && S.getASTContext().getTargetInfo().getTriple().isAArch64())
+return false;
Allen wrote:
> I find the a
Timm =?utf-8?q?Bäder?=
https://github.com/tbaederr review_requested
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
https://github.com/tbaederr review_requested
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
https://github.com/tbaederr review_requested
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/65462:
None
>From 20aa3e3225f6ea48df00ea05bcbfcc5cc2278af5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Tue, 8 Aug 2023 14:11:33 +0200
Subject: [PATCH 1/2] [clang][CFG]
Timm =?utf-8?q?Bäder?=
https://github.com/tbaederr review_requested
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
https://github.com/tbaederr review_requested
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
https://github.com/tbaederr edited
https://github.com/llvm/llvm-project/pull/65462
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jacquesguan marked an inline comment as done.
jacquesguan added a comment.
ping.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D158259/new/
https://reviews.llvm.org/D158259
___
cfe-commits mailing list
cf
https://github.com/dfszabo review_requested
https://github.com/llvm/llvm-project/pull/65465
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dfszabo review_requested
https://github.com/llvm/llvm-project/pull/65465
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dfszabo created
https://github.com/llvm/llvm-project/pull/65465:
GCC supports code like "asm volatile ("" : "=r" (i) : "0" (f))" where i is
integer type and f is floating point type. Currently this code produces an
error with Clang. The change allows mixed scalar types betwe
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65465
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hahnjo updated
https://github.com/llvm/llvm-project/pull/65257:
>From 7b52d2ad531286ca3e14c3f05da51c91fd71bd0d Mon Sep 17 00:00:00 2001
From: Jonas Hahnfeld
Date: Wed, 6 Sep 2023 13:11:57 +0200
Subject: [PATCH] [clang-repl] Emit const variables only once
Disable internal lin
https://github.com/DavidSpickett review_requested
https://github.com/llvm/llvm-project/pull/65466
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett created
https://github.com/llvm/llvm-project/pull/65466:
This follows the RISC-V work done in 4b40ced4e5ba10b841516b3970e7699ba8ded572.
This uses AArch64's target parser instead. We just list the names, without the
"+" on them, which matches RISC-V's format.
`
https://github.com/DavidSpickett review_requested
https://github.com/llvm/llvm-project/pull/65466
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett review_requested
https://github.com/llvm/llvm-project/pull/65466
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/github-actions[bot] labeled
https://github.com/llvm/llvm-project/pull/65466
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DavidSpickett review_requested
https://github.com/llvm/llvm-project/pull/65466
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
DavidSpickett wrote:
I've generalised this a bit but mostly kept to what RISC-V did. If we think
it's time to completely generalise it then I'll figure that out.
Mostly want to know people are on board with supporting this for AArch64 first.
https://github.com/llvm/llvm-project/pull/65466
hahnjo wrote:
The original patch worked for `clang-repl` but results in strong linkage which
I found to cause problems with modules downstream in ROOT. Instead the latest
push moves the special case one level higher to `basicGVALinkageForVariable`
and returns `GVA_DiscardableODR` which fixes t
1 - 100 of 446 matches
Mail list logo
