https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/158013
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
Duplicate of https://github.com/llvm/llvm-project/pull/158402.
https://github.com/llvm/llvm-project/pull/158403
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-br
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/158403
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic edited https://github.com/llvm/llvm-project/pull/158013
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/157810
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
TBH I don't think selects should be validated by profcheck at all, only
branches.
https://github.com/llvm/llvm-project/pull/157599
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
nikic wrote:
> This was merged to the release branch - the PR didn't update since I didn't
> have access to write to your repo @nikic (don't forget to check the "allow
> maintainer to write" checkbox in the future).
Huh, this is weird. I checked a few other PRs I have open, and they all have i
nikic wrote:
> > I don't know if I understand what you're going for in this patch. How is
> > this an improvement over just keeping REQUIRES: shell? The REQUIRES: bit
> > is what's going to stop the test from running in the wrong environment.
> > That seems like the right tradeoff.
>
> If we
https://github.com/nikic created
https://github.com/llvm/llvm-project/pull/157410
Backport of:
https://github.com/llvm/llvm-project/commit/a6148071300302b451ae92fbd8f8b955a6974891
https://github.com/llvm/llvm-project/commit/ec581e460ae92ef29c1ea4f200b36b79188fdd21
https://github.com/llvm/ll
https://github.com/nikic milestoned
https://github.com/llvm/llvm-project/pull/157410
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
As this test runs the whole optimization pipeline, please move it to
llvm/test/Transforms/PhaseOrdering.
https://github.com/llvm/llvm-project/pull/151649
___
llvm-branch-commits mailing list
llvm-branch-
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/151649
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic edited https://github.com/llvm/llvm-project/pull/151649
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic milestoned
https://github.com/llvm/llvm-project/pull/156815
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -697,8 +697,7 @@ static bool isSafeAndProfitableToSinkLoad(LoadInst *L) {
Instruction *InstCombinerImpl::foldPHIArgLoadIntoPHI(PHINode &PN) {
LoadInst *FirstLI = cast(PN.getIncomingValue(0));
- // Can't forward swifterror through a phi.
- if (FirstLI->getOperand(0)->isS
@@ -0,0 +1,167 @@
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
UTC_ARGS: --check-globals all --version 5
+; RUN: opt -passes=pre-isel-intrinsic-lowering -S < %s | FileCheck
--check-prefix=NOPAUTH %s
+; RUN: opt -passes=pre-isel-intrinsic-lowering -m
@@ -461,6 +463,162 @@ bool PreISelIntrinsicLowering::expandMemIntrinsicUses(
return Changed;
}
+namespace {
+
+enum class PointerEncoding {
+ Rotate,
+ PACCopyable,
+ PACNonCopyable,
+};
+
+bool expandProtectedFieldPtr(Function &Intr) {
+ Module &M = *Intr.getParent();
+
@@ -461,6 +463,162 @@ bool PreISelIntrinsicLowering::expandMemIntrinsicUses(
return Changed;
}
+namespace {
+
+enum class PointerEncoding {
+ Rotate,
+ PACCopyable,
+ PACNonCopyable,
+};
nikic wrote:
Unused?
https://github.com/llvm/llvm-project/pull/151
@@ -461,6 +463,162 @@ bool PreISelIntrinsicLowering::expandMemIntrinsicUses(
return Changed;
}
+namespace {
+
+enum class PointerEncoding {
+ Rotate,
+ PACCopyable,
+ PACNonCopyable,
+};
+
+bool expandProtectedFieldPtr(Function &Intr) {
+ Module &M = *Intr.getParent();
+
@@ -2850,6 +2850,12 @@ def int_experimental_convergence_anchor
def int_experimental_convergence_loop
: DefaultAttrsIntrinsic<[llvm_token_ty], [], [IntrNoMem, IntrConvergent]>;
+//===- Structure Protection Intrinsics
===//
+
+def int_prote
@@ -461,6 +463,162 @@ bool PreISelIntrinsicLowering::expandMemIntrinsicUses(
return Changed;
}
+namespace {
+
+enum class PointerEncoding {
+ Rotate,
+ PACCopyable,
+ PACNonCopyable,
+};
+
+bool expandProtectedFieldPtr(Function &Intr) {
+ Module &M = *Intr.getParent();
+
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/155513
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -634,7 +634,7 @@ class ASTContext : public RefCountedBase {
/// contain data that is address discriminated. This includes
/// implicitly authenticated values like vtable pointers, as well as
/// explicitly qualified fields.
- bool containsAddressDiscriminatedPointerAu
@@ -634,7 +634,7 @@ class ASTContext : public RefCountedBase {
/// contain data that is address discriminated. This includes
/// implicitly authenticated values like vtable pointers, as well as
/// explicitly qualified fields.
- bool containsAddressDiscriminatedPointerAu
@@ -553,16 +553,17 @@ void Instruction::dropUBImplyingAttrsAndUnknownMetadata(
}
void Instruction::dropUBImplyingAttrsAndMetadata(ArrayRef Keep) {
nikic wrote:
I think the `Keep` is no longer needed now, but I guess it doesn't hurt to keep
it...
https://git
@@ -1,20 +1,21 @@
-; NOTE: Assertions have been autogenerated by update_test_checks.py
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
UTC_ARGS: --check-globals all --version 5
nikic wrote:
I think you don't need the `--check-globals`,
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/154635
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic edited https://github.com/llvm/llvm-project/pull/154635
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -498,6 +498,7 @@ LLVM_ABI void dropDebugUsers(Instruction &I);
///
/// The moved instructions receive the insertion point debug location values
/// (DILocations) and their debug intrinsic instructions are removed.
+/// Selects and indirect calls keep their MD_prof metadata.
https://github.com/nikic requested changes to this pull request.
We need to also either backport a fix for lldb tests or disable the relevant
tests. That needs to happen as part of this PR.
https://github.com/llvm/llvm-project/pull/155251
___
llvm-bra
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/139423
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
You can probably achieve that just by adding a function call before the select
(where the function is not known willreturn/nounwind).
https://github.com/llvm/llvm-project/pull/152420
___
llvm-branch-comm
@@ -1678,6 +1680,8 @@ void
Instruction::dropUnknownNonDebugMetadata(ArrayRef KnownIDs) {
// A DIAssignID attachment is debug metadata, don't drop it.
KnownSet.insert(LLVMContext::MD_DIAssignID);
+ if (!ProfcheckDisableMetadataFixes)
+KnownSet.insert(LLVMContext::MD_p
nikic wrote:
All this tests needs is a hoistable select with prof metadata. You do not need
any of the blockaddress / indirectbr stuff.
https://github.com/llvm/llvm-project/pull/152420
___
llvm-branch-commits mailing
@@ -1678,6 +1680,8 @@ void
Instruction::dropUnknownNonDebugMetadata(ArrayRef KnownIDs) {
// A DIAssignID attachment is debug metadata, don't drop it.
KnownSet.insert(LLVMContext::MD_DIAssignID);
+ if (!ProfcheckDisableMetadataFixes)
+KnownSet.insert(LLVMContext::MD_p
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/152221
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
Closing this as LLVM 20 no longer accepts backports.
https://github.com/llvm/llvm-project/pull/148878
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/148878
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
Closing this as LLVM 20 no longer accepts backports.
https://github.com/llvm/llvm-project/pull/147554
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/147554
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
Closing this as LLVM 20 no longer accepts backports.
https://github.com/llvm/llvm-project/pull/147448
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/147448
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/142909
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
Closing this as LLVM 20 no longer accepts backports.
https://github.com/llvm/llvm-project/pull/142909
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
nikic wrote:
Closing this as LLVM 20 no longer accepts backports.
https://github.com/llvm/llvm-project/pull/137605
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/137605
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
Closing this as LLVM 20 no longer accepts backports.
https://github.com/llvm/llvm-project/pull/136863
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/136863
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
Closing this as LLVM 20 no longer accepts backports.
https://github.com/llvm/llvm-project/pull/133223
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commi
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/133223
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic requested changes to this pull request.
This is absolutely inappropriate to land on the release branch.
Feel free to update the version numbers -- or better, remove the mention of
specific versions entirely. These will get removed when we are ready to remove
them, not
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/152364
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -3490,19 +3514,25 @@ void SelectionDAGBuilder::visitCallBr(const CallBrInst
&I) {
// Update successor info.
addSuccessorWithProb(CallBrMBB, Return, BranchProbability::getOne());
- for (unsigned i = 0, e = I.getNumIndirectDests(); i < e; ++i) {
-BasicBlock *Dest =
@@ -9674,7 +9678,7 @@ This instruction requires several arguments:
indicates the function accepts a variable number of arguments, the
extra arguments can be specified.
#. '``fallthrough label``': the label reached when the inline assembly's
- execution exits the bottom.
nikic wrote:
I think it would be a good idea to land the refactorings here (which just split
things into separate methods) as a separate NFC change. It's kind of hard to
follow what's going on here with all the code movements mixed in.
https://github.com/llvm
@@ -3006,10 +3024,41 @@ bool IRTranslator::translateInvoke(const User &U,
return true;
}
+/// The intrinsics currently supported by callbr are implicit control flow
+/// intrinsics such as amdgcn.kill.
bool IRTranslator::translateCallBr(const User &U,
@@ -359,6 +359,11 @@ UseCaptureInfo llvm::DetermineUseCaptureKind(const Use &U,
const Value *Base) {
case Instruction::AddrSpaceCast:
// The original value is not captured via this if the new value isn't.
return UseCaptureInfo::passthrough();
+ case Instruction::Ptr
nikic wrote:
Should also add an entry in
https://llvm.org/docs/LangRef.html#constant-expressions.
https://github.com/llvm/llvm-project/pull/139357
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.o
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/139357
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -12521,6 +12521,59 @@ Example:
%Y = ptrtoint ptr %P to i64; yields zero
extension on 32-bit architecture
%Z = ptrtoint <4 x ptr> %P to <4 x i64>; yields vector zero extension
for a vector of addresses on 32-bit architecture
+.. _i_ptrto
@@ -3532,6 +3533,28 @@ void Verifier::visitFPToSIInst(FPToSIInst &I) {
visitInstruction(I);
}
+void Verifier::visitPtrToAddrInst(PtrToAddrInst &I) {
+ // Get the source and destination types
+ Type *SrcTy = I.getOperand(0)->getType();
+ Type *DestTy = I.getType();
+
+ Ch
https://github.com/nikic edited https://github.com/llvm/llvm-project/pull/139357
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
@@ -731,6 +731,12 @@ class TargetTransformInfoImplBase {
return 0;
break;
}
+case Instruction::PtrToAddr: {
+ unsigned DstSize = Dst->getScalarSizeInBits();
+ if (DL.isLegalInteger(DstSize) && DstSize >=
DL.getAddressSizeInBits(Src))
---
@@ -12521,6 +12521,59 @@ Example:
%Y = ptrtoint ptr %P to i64; yields zero
extension on 32-bit architecture
%Z = ptrtoint <4 x ptr> %P to <4 x i64>; yields vector zero extension
for a vector of addresses on 32-bit architecture
+.. _i_ptrto
@@ -3532,6 +3533,28 @@ void Verifier::visitFPToSIInst(FPToSIInst &I) {
visitInstruction(I);
}
+void Verifier::visitPtrToAddrInst(PtrToAddrInst &I) {
+ // Get the source and destination types
+ Type *SrcTy = I.getOperand(0)->getType();
+ Type *DestTy = I.getType();
+
+ Ch
@@ -1482,6 +1482,20 @@ Constant *llvm::ConstantFoldCastOperand(unsigned Opcode,
Constant *C,
switch (Opcode) {
default:
llvm_unreachable("Missing case");
+ case Instruction::PtrToAddr:
+if (auto *GEP = dyn_cast(C)) {
+ // For now just handle the basic case of
@@ -12521,6 +12521,59 @@ Example:
%Y = ptrtoint ptr %P to i64; yields zero
extension on 32-bit architecture
%Z = ptrtoint <4 x ptr> %P to <4 x i64>; yields vector zero extension
for a vector of addresses on 32-bit architecture
+.. _i_ptrto
https://github.com/nikic approved this pull request.
LGTM, though I wouldn't expect to see much usub.with.overflow in the middle-end.
https://github.com/llvm/llvm-project/pull/135785
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.o
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/152099
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
That sounds fine as long as the plan is to remove the isNonIntegralPointer()
method in the future, in favor of more precise checks.
https://github.com/llvm/llvm-project/pull/105735
___
llvm-branch-commits mailing list
llvm-branch-commits@
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/148782
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
This looks like a simple missed optimization in InstCombine.
My first thought would be to handle this via foldOpIntoPhi for selects:
https://github.com/nikic/llvm-project/commit/c7d9b006453275be94894c6afdae5f6c940c1621
But this isn't quite correc
nikic wrote:
Ah, I didn't realize that it's always based on op_end.
Maybe this change is not actually that bad if combined with the new
successors() representation? Then the successors iterator is not going to
through through getSuccessor() anymore and it will have less impact?
https://github
https://github.com/nikic approved this pull request.
LGTM, though I'd probably squash the first two commits. The back and forth is
confusing.
https://github.com/llvm/llvm-project/pull/150425
___
llvm-branch-commits mailing list
llvm-branch-commits@lis
@@ -461,6 +465,198 @@ bool PreISelIntrinsicLowering::expandMemIntrinsicUses(
return Changed;
}
+namespace {
+
+enum class PointerEncoding {
+ Rotate,
+ PACCopyable,
+ PACNonCopyable,
+};
+
+bool expandProtectedFieldPtr(Function &Intr) {
+ Module &M = *Intr.getParent();
+
nikic wrote:
Use update_test_checks.py.
https://github.com/llvm/llvm-project/pull/151647
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bra
@@ -0,0 +1,46 @@
+; RUN: opt -passes=pre-isel-intrinsic-lowering -S < %s | FileCheck
--check-prefixes=CHECK,NOPAUTH %s
+; RUN: opt -passes=pre-isel-intrinsic-lowering -mattr=+pauth -S < %s |
FileCheck --check-prefixes=CHECK,PAUTH %s
+
+target triple = "aarch64-unknown-linux-gnu"
@@ -37,6 +39,8 @@
#include "llvm/Transforms/Utils/LowerMemIntrinsics.h"
#include "llvm/Transforms/Utils/LowerVectorIntrinsics.h"
+#include
nikic wrote:
Do not use `std::set` unless you actually need an ordered set. Based on usage,
you want SmallPtrSet.
htt
@@ -31161,3 +31161,57 @@ This intrinsic is assumed to execute in the default
:ref:`floating-point
environment ` *except* for the rounding mode.
This intrinsic is not supported on all targets. Some targets may not support
all rounding modes.
+
+'``llvm.protected.field.ptr``' In
@@ -461,6 +465,198 @@ bool PreISelIntrinsicLowering::expandMemIntrinsicUses(
return Changed;
}
+namespace {
+
+enum class PointerEncoding {
+ Rotate,
+ PACCopyable,
+ PACNonCopyable,
+};
+
+bool expandProtectedFieldPtr(Function &Intr) {
+ Module &M = *Intr.getParent();
+
@@ -0,0 +1,33 @@
+; RUN: opt -O2 -S < %s | FileCheck %s
+
+; Test that no optimization run at -O2 moves the loads into the exit block,
+; as this causes unnecessary address escapes with pointer field protection.
+
+target triple = "aarch64-unknown-linux-gnu"
nikic
@@ -3846,10 +3846,7 @@ bool llvm::canReplaceOperandWithVariable(const
Instruction *I, unsigned OpIdx) {
if (Op->getType()->isMetadataTy())
return false;
- // swifterror pointers can only be used by a load, store, or as a swifterror
- // argument; swifterror pointers a
nikic wrote:
Use update_test_checks.py.
https://github.com/llvm/llvm-project/pull/151649
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-bra
nikic wrote:
Use update_test_checks.py.
Please also add a test where the alloca is split but not promoted.
https://github.com/llvm/llvm-project/pull/151650
___
llvm-branch-commits mailing list
llvm-branch-commits@lis
@@ -0,0 +1,41 @@
+; RUN: opt -passes=sroa -S < %s | FileCheck %s
+
+target triple = "aarch64-unknown-linux-gnu"
nikic wrote:
Is this triple necessary? If not, drop it, otherwise add REQUIRES.
https://github.com/llvm/llvm-project/pull/151650
_
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/133535
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic commented:
LGTM
Something I don't entirely like is that MIMetadata holds DebugLoc, but
copyMIMetadata does not copy it -- instead, it needs to be passed to
CreateMachineInstr. But given how this is used it's not really a problem...
https://github.com/llvm/llvm-project
nikic wrote:
> I'm not actively aware of any cases where the non-integrality is being used
> to block invalid optimizations ... but I do know that, for example,
> SeparateConstOffsetFromGEP has a mode (which I think Nvidia uses) that turns
> pointer arithmetic into ptrtoint/inttoptr pairs.
Th
nikic wrote:
Looks like the release binaries jobs are failing.
https://github.com/llvm/llvm-project/pull/151245
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
> I would be happy to change this to just unstable and external state. I'm not
> particularly happy with this name but I can't come up with something better.
I think we should do that.
https://github.com/llvm/llvm-project/pull/105735
___
nikic wrote:
Yes, I feel strongly that this should not be backported. This patch is both
unimportant and risky.
https://github.com/llvm/llvm-project/pull/150771
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm
Author: Nikita Popov
Date: 2025-07-28T10:02:10+02:00
New Revision: a268b616a622b20056c908c53aa762d1fc5402f1
URL:
https://github.com/llvm/llvm-project/commit/a268b616a622b20056c908c53aa762d1fc5402f1
DIFF:
https://github.com/llvm/llvm-project/commit/a268b616a622b20056c908c53aa762d1fc5402f1.diff
Author: Nikita Popov
Date: 2025-07-28T09:52:06+02:00
New Revision: 4a2a3608513404b7005f96c1b4476f91d34f01de
URL:
https://github.com/llvm/llvm-project/commit/4a2a3608513404b7005f96c1b4476f91d34f01de
DIFF:
https://github.com/llvm/llvm-project/commit/4a2a3608513404b7005f96c1b4476f91d34f01de.diff
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/150746
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic milestoned
https://github.com/llvm/llvm-project/pull/150746
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic requested changes to this pull request.
I don't think this should be backported.
https://github.com/llvm/llvm-project/pull/150771
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cg
https://github.com/nikic milestoned
https://github.com/llvm/llvm-project/pull/150771
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/150646
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
nikic wrote:
@vvereschaka That issue only needs the change to the sort predicate, not this
whole PR.
https://github.com/llvm/llvm-project/pull/150192
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
nikic wrote:
Do we need all of those? For the release branch, can we revert the change
fully? Or is other stuff already based on it?
https://github.com/llvm/llvm-project/pull/150425
___
llvm-branch-commits mailing list
llvm-branch-commits@lists.llvm.o
1 - 100 of 660 matches
Mail list logo
