https://github.com/isuckatcs updated
https://github.com/llvm/llvm-project/pull/70053
>From 7cca7a3a6d969318fb8531751f75bb41715c7475 Mon Sep 17 00:00:00 2001
From: isuckatcs <65320245+isucka...@users.noreply.github.com>
Date: Sat, 30 Sep 2023 17:05:02 +0200
Subject: [PATCH 1/4] cleanup
---
@@ -54,7 +54,7 @@ class ByteCodeExprGen : public
ConstStmtVisitor, bool>,
public:
/// Initializes the compiler and the backend emitter.
template
- ByteCodeExprGen(Context , Program , Tys &&... Args)
+ ByteCodeExprGen(Context , Program , Tys &&...Args)
https://github.com/ilinpv approved this pull request.
LGTM, thanks for additinal FMV tests!
https://github.com/llvm/llvm-project/pull/79455
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/Endilll closed
https://github.com/llvm/llvm-project/pull/67948
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Vlad Serebrennikov
Date: 2024-01-25T19:00:17+04:00
New Revision: 43e46c546a1f0c7c3e436104a974560c6a2e5188
URL:
https://github.com/llvm/llvm-project/commit/43e46c546a1f0c7c3e436104a974560c6a2e5188
DIFF:
https://github.com/isuckatcs updated
https://github.com/llvm/llvm-project/pull/70053
>From 7cca7a3a6d969318fb8531751f75bb41715c7475 Mon Sep 17 00:00:00 2001
From: isuckatcs <65320245+isucka...@users.noreply.github.com>
Date: Sat, 30 Sep 2023 17:05:02 +0200
Subject: [PATCH 1/4] cleanup
---
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Alexandros Lamprineas (labrinea)
Changes
Adds tests showing that we select function version according to the highest
feature priority. This will make the changes introduced by #79316 more
evident.
---
Full diff:
https://github.com/labrinea created
https://github.com/llvm/llvm-project/pull/79455
Adds tests showing that we select function version according to the highest
feature priority. This will make the changes introduced by #79316 more evident.
>From 07719ca748ad94e3c458293e46538491d771cc00 Mon
@@ -54,7 +54,7 @@ class ByteCodeExprGen : public
ConstStmtVisitor, bool>,
public:
/// Initializes the compiler and the backend emitter.
template
- ByteCodeExprGen(Context , Program , Tys &&... Args)
+ ByteCodeExprGen(Context , Program , Tys &&...Args)
https://github.com/isuckatcs updated
https://github.com/llvm/llvm-project/pull/70053
>From 7cca7a3a6d969318fb8531751f75bb41715c7475 Mon Sep 17 00:00:00 2001
From: isuckatcs <65320245+isucka...@users.noreply.github.com>
Date: Sat, 30 Sep 2023 17:05:02 +0200
Subject: [PATCH 1/3] cleanup
---
https://github.com/isuckatcs edited
https://github.com/llvm/llvm-project/pull/70053
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -54,7 +54,7 @@ class ByteCodeExprGen : public
ConstStmtVisitor, bool>,
public:
/// Initializes the compiler and the backend emitter.
template
- ByteCodeExprGen(Context , Program , Tys &&... Args)
+ ByteCodeExprGen(Context , Program , Tys &&...Args)
@@ -2694,6 +2852,9 @@ class VPlan {
/// been modeled in VPlan directly.
DenseMap SCEVToExpansion;
+ /// Construct an uninitialized VPlan, should be used for cloning only.
+ explicit VPlan() = default;
+
ayalz wrote:
Is it really needed? (See above)
@@ -982,6 +1037,94 @@ void VPlan::updateDominatorTree(DominatorTree *DT,
BasicBlock *LoopHeaderBB,
assert(DT->verify(DominatorTree::VerificationLevel::Fast));
}
+static void remapOperands(VPBlockBase *Entry, VPBlockBase *NewEntry,
+ DenseMap ) {
+
@@ -982,6 +1037,94 @@ void VPlan::updateDominatorTree(DominatorTree *DT,
BasicBlock *LoopHeaderBB,
assert(DT->verify(DominatorTree::VerificationLevel::Fast));
}
+static void remapOperands(VPBlockBase *Entry, VPBlockBase *NewEntry,
+ DenseMap ) {
+
@@ -614,6 +614,61 @@ void VPBasicBlock::print(raw_ostream , const Twine
,
printSuccessors(O, Indent);
}
#endif
+static void cloneCFG(VPBlockBase *Entry,
+ DenseMap );
+
+static VPBlockBase *cloneVPB(VPBlockBase *BB) {
+ if (auto *VPBB = dyn_cast(BB)) {
@@ -614,6 +614,61 @@ void VPBasicBlock::print(raw_ostream , const Twine
,
printSuccessors(O, Indent);
}
#endif
+static void cloneCFG(VPBlockBase *Entry,
+ DenseMap );
+
+static VPBlockBase *cloneVPB(VPBlockBase *BB) {
+ if (auto *VPBB = dyn_cast(BB)) {
@@ -982,6 +1037,94 @@ void VPlan::updateDominatorTree(DominatorTree *DT,
BasicBlock *LoopHeaderBB,
assert(DT->verify(DominatorTree::VerificationLevel::Fast));
}
+static void remapOperands(VPBlockBase *Entry, VPBlockBase *NewEntry,
+ DenseMap ) {
+
@@ -982,6 +1037,94 @@ void VPlan::updateDominatorTree(DominatorTree *DT,
BasicBlock *LoopHeaderBB,
assert(DT->verify(DominatorTree::VerificationLevel::Fast));
}
+static void remapOperands(VPBlockBase *Entry, VPBlockBase *NewEntry,
+ DenseMap ) {
+
@@ -1594,6 +1657,13 @@ class VPWidenPHIRecipe : public VPHeaderPHIRecipe {
addOperand(Start);
}
+ VPRecipeBase *clone() override {
+auto *Res = new VPWidenPHIRecipe(cast(getUnderlyingInstr()),
ayalz wrote:
Better mark it unreachable than have
@@ -614,6 +614,61 @@ void VPBasicBlock::print(raw_ostream , const Twine
,
printSuccessors(O, Indent);
}
#endif
+static void cloneCFG(VPBlockBase *Entry,
+ DenseMap );
ayalz wrote:
This cloning is recursive, so perhaps more accurately
@@ -614,6 +614,61 @@ void VPBasicBlock::print(raw_ostream , const Twine
,
printSuccessors(O, Indent);
}
#endif
+static void cloneCFG(VPBlockBase *Entry,
+ DenseMap );
+
+static VPBlockBase *cloneVPB(VPBlockBase *BB) {
ayalz wrote:
Author: Alexandre Ganea
Date: 2024-01-25T09:34:17-05:00
New Revision: ee15e2bd32a4677c40d927c732b26d33f88d7865
URL:
https://github.com/llvm/llvm-project/commit/ee15e2bd32a4677c40d927c732b26d33f88d7865
DIFF:
Author: Alexandre Ganea
Date: 2024-01-25T09:34:17-05:00
New Revision: f33f5a04e9feeb9b473694825d84da8322f87df9
URL:
https://github.com/llvm/llvm-project/commit/f33f5a04e9feeb9b473694825d84da8322f87df9
DIFF:
Author: Alexandre Ganea
Date: 2024-01-25T09:34:17-05:00
New Revision: 419d6ea135dd205e1eaab368a58ae14f9f52f699
URL:
https://github.com/llvm/llvm-project/commit/419d6ea135dd205e1eaab368a58ae14f9f52f699
DIFF:
https://github.com/tbaederr unassigned
https://github.com/llvm/llvm-project/pull/79452
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr unassigned
https://github.com/llvm/llvm-project/pull/79452
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr unassigned
https://github.com/llvm/llvm-project/pull/79452
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr unassigned
https://github.com/llvm/llvm-project/pull/79452
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4216,6 +4217,97 @@ MachineSDNode *X86DAGToDAGISel::emitPCMPESTR(unsigned
ROpc, unsigned MOpc,
return CNode;
}
+// When the consumer of a right shift (arithmetic or logical) wouldn't notice
+// the difference if the instruction was a rotate right instead (because the
https://github.com/RKSimon requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/77964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4216,6 +4217,97 @@ MachineSDNode *X86DAGToDAGISel::emitPCMPESTR(unsigned
ROpc, unsigned MOpc,
return CNode;
}
+// When the consumer of a right shift (arithmetic or logical) wouldn't notice
+// the difference if the instruction was a rotate right instead (because the
@@ -4216,6 +4217,97 @@ MachineSDNode *X86DAGToDAGISel::emitPCMPESTR(unsigned
ROpc, unsigned MOpc,
return CNode;
}
+// When the consumer of a right shift (arithmetic or logical) wouldn't notice
+// the difference if the instruction was a rotate right instead (because the
https://github.com/RKSimon edited
https://github.com/llvm/llvm-project/pull/77964
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kleinesfilmroellchen wrote:
> It's sad that there's not a single codepath that would handle all these, but
> maybe the perfect is the enemy of the good here.
The code duplication was my reason for looking at the common path. If you're
fine with duplicating the code I will go ahead and do
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
In C++, we get a ComplexToBool cast, but we might not in C.
---
Full diff: https://github.com/llvm/llvm-project/pull/79452.diff
4 Files Affected:
- (modified) clang/lib/AST/Interp/ByteCodeExprGen.cpp
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/79452
In C++, we get a ComplexToBool cast, but we might not in C.
>From b207bff4cda462869bcf8547e44d23e5b8fc5e94 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 25 Jan 2024 15:20:52 +0100
@@ -211,10 +214,12 @@ class IncludeInserter {
// include path of non-verbatim header will not be shortened.
IncludeInserter(StringRef FileName, StringRef Code,
const format::FormatStyle , StringRef BuildDir,
- HeaderSearch
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Tor Shepherd (torshepherd)
Changes
This PR attempts to fix [1536.](https://github.com/clangd/clangd/issues/1536).
See in the unit tests, when all quick fixes are of the same `code`,
`isPreferred` will be true. However, this doesn't seem
github-actions[bot] wrote:
Thank you for submitting a Pull Request (PR) to the LLVM Project!
This PR will be automatically labeled and the relevant teams will be
notified.
If you wish to, you can add reviewers by using the "Reviewers" section on this
page.
If this is not working for you, it
https://github.com/torshepherd created
https://github.com/llvm/llvm-project/pull/79448
This PR attempts to fix [1536.](https://github.com/clangd/clangd/issues/1536).
See in the unit tests, when all quick fixes are of the same `code`,
`isPreferred` will be true. However, this doesn't seem to
@@ -296,6 +296,19 @@ struct Fragment {
// ::). All nested namespaces are affected as well.
// Affects availability of the AddUsing tweak.
std::vector> FullyQualifiedNamespaces;
+
+/// List of regexes for headers that should always be included with a
+///
erichkeane wrote:
>As I understood the author, more time was needed to prepare the tests and the
>fix. I just tried to bring the tip of the tree to a good state soon (our
>internal release process was blocked on this) without putting any pressure on
>the author.
Typically when the author is
@@ -49,6 +49,11 @@ constexpr uint32_t VersionMajorV5 = 1;
/// HSA metadata minor version for code object V5.
constexpr uint32_t VersionMinorV5 = 2;
+/// HSA metadata major version for code object V6.
+constexpr uint32_t VersionMajorV6 = 1;
+/// HSA metadata minor version for
SamTebbs33 wrote:
Agreed. I tried moving it to Support but that introduced a dependency cycle, so
I re-implemented some of the SMEAttributes logic in-place instead.
https://github.com/llvm/llvm-project/pull/77936
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/79373
>From 145b7bc932ce3ffa46545cd7af29b1c93981429c Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 24 Jan 2024 15:34:00 -0600
Subject: [PATCH 1/3] [NVPTX] Add support for -march=native in standalone NVPTX
https://github.com/jhuber6 updated
https://github.com/llvm/llvm-project/pull/79373
>From 145b7bc932ce3ffa46545cd7af29b1c93981429c Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Wed, 24 Jan 2024 15:34:00 -0600
Subject: [PATCH 1/2] [NVPTX] Add support for -march=native in standalone NVPTX
@@ -851,6 +851,28 @@ static void InitializePredefinedMacros(const TargetInfo
,
Twine(getClangFullCPPVersion()) + "\"");
// Initialize language-specific preprocessor defines.
+ if (LangOpts.getStdlibHardeningMode()) {
+const char
https://github.com/SLTozer approved this pull request.
LGTM with latest changes.
https://github.com/llvm/llvm-project/pull/77930
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4266,19 +4285,140 @@ bool
InstCombinerImpl::tryToSinkInstruction(Instruction *I,
// Perform salvaging without the clones, then sink the clones.
if (!DIIClones.empty()) {
-// RemoveDIs: pass in empty vector of DPValues until we get to
instrumenting
-// this
jhuber6 wrote:
> I think I'm with Art on this one.
>
> > > Problem #2 [...] The arch=native will create a working configuration, but
> > > would build more than necessary.
> >
> >
> > It will target the first GPU it finds. We could maybe change the behavior
> > to detect the newest, but the
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: None (NagyDonat)
Changes
Previously the function `RangeConstraintManager::printValue()` crashed when it
encountered an empty rangeset (because `RangeSet::getBitwidth()` and
`RangeSet::isUnsigned()` assert that the
chenzheng1030 wrote:
Patch updated.
https://github.com/llvm/llvm-project/pull/79109
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NagyDonat created
https://github.com/llvm/llvm-project/pull/79446
Previously the function `RangeConstraintManager::printValue()` crashed when it
encountered an empty rangeset (because `RangeSet::getBitwidth()` and
`RangeSet::isUnsigned()` assert that the rangeset is not
https://github.com/chenzheng1030 updated
https://github.com/llvm/llvm-project/pull/79109
>From 014b10f43e2d3f8564940e21033cee77c3c0c10e Mon Sep 17 00:00:00 2001
From: Nemanja Ivanovic
Date: Tue, 23 Jan 2024 03:25:01 -0500
Subject: [PATCH 1/2] [PowerPC] Diagnose invalid combination with
jyknight wrote:
Does this cause an ABI incompatibility? E.g. if we have a case where an
existing object calls copyStruct on a given object (which presumably has its
own internal mutex), and a newly compiled object file calls `__atomic_load` on
the same object, implemented with its own
https://github.com/mariusz-sikora-at-amd updated
https://github.com/llvm/llvm-project/pull/78729
>From 56cf06f1b530d5ec62de1cc3818bf2f76dfd Mon Sep 17 00:00:00 2001
From: Mariusz Sikora
Date: Fri, 19 Jan 2024 16:29:46 +0100
Subject: [PATCH] [AMDGPU][GFX12] Add tests for unsupported
Author: Timm Bäder
Date: 2024-01-25T14:21:59+01:00
New Revision: cd0b0055a730e55f2f14f35172e05dc27642f8ce
URL:
https://github.com/llvm/llvm-project/commit/cd0b0055a730e55f2f14f35172e05dc27642f8ce
DIFF:
https://github.com/llvm/llvm-project/commit/cd0b0055a730e55f2f14f35172e05dc27642f8ce.diff
https://github.com/H-G-Hristov updated
https://github.com/llvm/llvm-project/pull/79032
>From e03452fda84a5284420bba1913299b68caabb6cd Mon Sep 17 00:00:00 2001
From: Zingam
Date: Mon, 22 Jan 2024 20:35:00 +0200
Subject: [PATCH 1/2] Revert "Revert "[libc++][format] P2637R3: Member `visit`
https://github.com/4vtomat approved this pull request.
Thanks, LGTM~
https://github.com/llvm/llvm-project/pull/77889
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/spaits updated
https://github.com/llvm/llvm-project/pull/79371
From d4ca5c2fcb87f424be23efc4513df491403c3811 Mon Sep 17 00:00:00 2001
From: Gabor Spaits
Date: Wed, 24 Jan 2024 21:21:26 +0100
Subject: [PATCH 1/7] [Sema]Substitue template parameter packs when deduced
from
@@ -380,14 +380,14 @@ void RISCVIntrinsicManagerImpl::InitRVVIntrinsic(
OverloadedName += "_" + OverloadedSuffixStr.str();
// clang built-in function name, e.g. __builtin_rvv_vadd.
- std::string BuiltinName = "__builtin_rvv_" + std::string(Record.Name);
+ std::string
https://github.com/4vtomat updated
https://github.com/llvm/llvm-project/pull/77487
>From 84ea759c43d8e9cb450d95d00fd802be622153a2 Mon Sep 17 00:00:00 2001
From: Brandon Wu
Date: Sun, 7 Jan 2024 18:10:59 -0800
Subject: [PATCH 1/2] [RISCV][clang] Optimize memory usage of intrinsic lookup
table
NagyDonat wrote:
Yes, the "common big set of arguments" is an annoying problem. Perhaps you
could consider my solution in core.BitwiseShift where I introduced a
"validator" class that holds all the common arguments as data members, so
instead of helper functions you have helper methods that
Author: Timm Bäder
Date: 2024-01-25T13:29:39+01:00
New Revision: f4ed7f8d0a3830d05e53476fc64966e871bf9454
URL:
https://github.com/llvm/llvm-project/commit/f4ed7f8d0a3830d05e53476fc64966e871bf9454
DIFF:
https://github.com/llvm/llvm-project/commit/f4ed7f8d0a3830d05e53476fc64966e871bf9454.diff
@@ -431,6 +442,17 @@ namespace deduction_after_explicit_pack {
i(0, 1, 2, 3, 4, 5); // expected-error {{no match}}
}
+ template
+ void bar(args_tag, type_identity_t..., int mid,
type_identity_t...) {}
spaits wrote:
It could work. It would cost only
balazske wrote:
My concern was the addition of many small functions that are used only from few
other `eval*` calls and are relatively special. And all of these need a common
big set of arguments like `StreamSym`, `CE`, `Call`. The build of states for
success and failure cases in the thing
mrexodia wrote:
> I haven't checked closely yet, but it seems like you need to add tests.
Could you provide some guidance on what kind of tests to add and how to
actually run them locally? First I wanted to get the builtkite job to be green,
but it seems to be failing on some unrelated test
@@ -21471,6 +21471,53 @@ bool isHalvingTruncateOfLegalScalableType(EVT SrcVT,
EVT DstVT) {
(SrcVT == MVT::nxv2i64 && DstVT == MVT::nxv2i32);
}
+// Combine store (trunc X to <3 x i8>) to sequence of ST1.b.
+static SDValue combineI8TruncStore(StoreSDNode *ST,
@@ -40,6 +41,7 @@ using namespace llvm;
namespace lld::wasm {
static std::unique_ptr createLTO() {
lto::Config c;
+ c.EmbedCmdArgs = commonContext().cmdArgs;
mrexodia wrote:
I changed all the `commonContext()` to `context()` (or the `ctx` variable
https://github.com/mrexodia updated
https://github.com/llvm/llvm-project/pull/79390
>From 67280cb8a77931271e685f7c92718d45cfea69a8 Mon Sep 17 00:00:00 2001
From: Duncan Ogilvie
Date: Thu, 25 Jan 2024 00:08:49 +0100
Subject: [PATCH] Embed the command line arguments during LTO
---
https://github.com/fhahn updated https://github.com/llvm/llvm-project/pull/78637
>From efd07e93aed51049ad3783c701284617ae446330 Mon Sep 17 00:00:00 2001
From: Florian Hahn
Date: Wed, 17 Jan 2024 11:11:59 +
Subject: [PATCH 1/7] [AArch64] Combine store (trunc X to <3 x i8>) to sequence
of
chapuni wrote:
Seems this causes the crash with `-fcoverage-mcdc -mllvm
-system-headers-coverage`. Investigating.
See also #78920.
https://github.com/llvm/llvm-project/pull/76950
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -860,7 +841,7 @@ bool
AArch64TargetInfo::handleTargetFeatures(std::vector ,
}
if (Feature == "+dit")
HasDIT = true;
-if (Feature == "+cccp")
+if (Feature == "+ccpp")
ostannard wrote:
Again, does this need a test?
@@ -741,7 +722,7 @@ bool
AArch64TargetInfo::handleTargetFeatures(std::vector ,
if (Feature == "+neon" || Feature == "+fp-armv8")
FPU |= NeonMode;
-if (Feature == "+jscvt") {
+if (Feature == "+jsconv") {
ostannard wrote:
This looks like bug
@@ -498,10 +498,34 @@ static VPValue *createScalarIVSteps(VPlan , const
InductionDescriptor ,
VPCanonicalIVPHIRecipe *CanonicalIV = Plan.getCanonicalIV();
Type *TruncTy = TruncI ? TruncI->getType() : IVTy;
VPValue *BaseIV = CanonicalIV;
fhahn wrote:
@@ -1469,6 +1461,52 @@ void VPReplicateRecipe::print(raw_ostream , const
Twine ,
}
#endif
+static bool isUniformAcrossVFsAndUFs(VPScalarCastRecipe *C) {
+ return C->isDefinedOutsideVectorRegions() ||
+ isa(C->getOperand(0)) ||
+ isa(C->getOperand(0));
+}
+
https://github.com/fhahn commented:
Comments should be addressed and title & description updated, thanks!
https://github.com/llvm/llvm-project/pull/78113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
@@ -230,7 +230,11 @@ Type *VPTypeAnalysis::inferScalarType(const VPValue *V) {
return V->getUnderlyingValue()->getType();
})
.Case(
- [](const VPWidenCastRecipe *R) { return R->getResultType(); });
+ [](const
https://github.com/fhahn edited https://github.com/llvm/llvm-project/pull/78113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1469,6 +1461,52 @@ void VPReplicateRecipe::print(raw_ostream , const
Twine ,
}
#endif
+static bool isUniformAcrossVFsAndUFs(VPScalarCastRecipe *C) {
fhahn wrote:
Added comment + TODO, thanks!
https://github.com/llvm/llvm-project/pull/78113
https://github.com/fhahn updated https://github.com/llvm/llvm-project/pull/78113
>From 36b085f21b76d7bf7c9965a86a09d1cef4fe9329 Mon Sep 17 00:00:00 2001
From: Florian Hahn
Date: Sun, 14 Jan 2024 14:13:08 +
Subject: [PATCH 1/6] [VPlan] Add new VPUniformPerUFRecipe, use for step
truncation.
https://github.com/fhahn edited https://github.com/llvm/llvm-project/pull/78113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -19520,6 +19520,13 @@ static bool actOnOMPReductionKindClause(
bool FirstIter = true;
for (Expr *RefExpr : VarList) {
assert(RefExpr && "nullptr expr in OpenMP reduction clause.");
+if (ClauseKind == OMPC_reduction &&
+RD.RedModifier ==
https://github.com/fhahn edited https://github.com/llvm/llvm-project/pull/78113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
steakhal wrote:
> I'm seconding the suggestions of @steakhal, and in particular I agree with
>
> > I'd also advise against using more callables bundled with CallDescriptions.
> > They make debugging code more difficult, as the control-flow would become
> > also data-dependent. I'd suggest
NagyDonat wrote:
I'm seconding the suggestions of @steakhal, and in particular I agree with
> I'd also advise against using more callables bundled with CallDescriptions.
> They make debugging code more difficult, as the control-flow would become
> also data-dependent. I'd suggest other ways to
https://github.com/spaits updated
https://github.com/llvm/llvm-project/pull/79371
From d4ca5c2fcb87f424be23efc4513df491403c3811 Mon Sep 17 00:00:00 2001
From: Gabor Spaits
Date: Wed, 24 Jan 2024 21:21:26 +0100
Subject: [PATCH 1/6] [Sema]Substitue template parameter packs when deduced
from
@@ -12,28 +12,6 @@
void foo() {}
-template
saiislam wrote:
How was this test case working till now when templates were not supported in
scan?
https://github.com/llvm/llvm-project/pull/79431
___
cfe-commits
martinboehme wrote:
> > You mean the publicness of `getResultObjectLocation()`?
>
> Yeah, I was wondering if it would be more user friendly if something like
> `State.Env.get(Expr);` would automatically recognize
> that the user actually wants the result location and returned that without
>
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/66514
>From 2a0b02a285203228944d5dd206f968e776757993 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 15 Sep 2023 15:51:39 +0200
Subject: [PATCH] [clang][Diagnostics] Highlight code snippets
@@ -389,10 +390,29 @@ template class LocalScope : public
VariableScope {
if (!Local.Desc->isPrimitive() && !Local.Desc->isPrimitiveArray()) {
this->Ctx->emitGetPtrLocal(Local.Offset, SourceInfo{});
this->Ctx->emitRecordDestruction(Local.Desc);
+
@@ -54,7 +54,7 @@ class ByteCodeExprGen : public
ConstStmtVisitor, bool>,
public:
/// Initializes the compiler and the backend emitter.
template
- ByteCodeExprGen(Context , Program , Tys &&... Args)
+ ByteCodeExprGen(Context , Program , Tys &&...Args)
@@ -389,10 +390,29 @@ template class LocalScope : public
VariableScope {
if (!Local.Desc->isPrimitive() && !Local.Desc->isPrimitiveArray()) {
this->Ctx->emitGetPtrLocal(Local.Offset, SourceInfo{});
this->Ctx->emitRecordDestruction(Local.Desc);
+
https://github.com/spaits edited https://github.com/llvm/llvm-project/pull/79371
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/79398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -44,7 +47,9 @@ class BugSuppression {
using CachedRanges =
llvm::SmallVector;
- llvm::DenseMap CachedSuppressionLocations;
+ llvm::DenseMap CachedSuppressionLocations{};
+
+ ASTContext
steakhal wrote:
```suggestion
llvm::DenseMap
@@ -27,6 +28,8 @@ class PathDiagnosticLocation;
class BugSuppression {
public:
+ BugSuppression(ASTContext ) : ACtx(ACtx) {}
steakhal wrote:
```suggestion
explicit BugSuppression(const ASTContext ) : ACtx(ACtx) {}
```
https://github.com/steakhal commented:
I only have minor nits. No objections.
https://github.com/llvm/llvm-project/pull/79398
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -4371,6 +4397,41 @@ Sema::TemplateDeductionResult
Sema::DeduceTemplateArguments(
// corresponding argument is a list?
PackScope.nextPackElement();
}
+ } else if (!IsTrailingPack && !PackScope.isPartiallyExpanded() &&
+
401 - 500 of 548 matches
Mail list logo