[llvm] [WIP][Don't Merge] Check whether lines are covered by existing tests (PR #184637)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 4 07:49:36 PST 2026
https://github.com/ambergorzynski created https://github.com/llvm/llvm-project/pull/184637
We are confirming that certain lines are not covered by any existing tests. We will submit corresponding tests for each case once we have confirmed this.
>From eb2baa50d5c06254b76ae586fdb17e73f6e654cf Mon Sep 17 00:00:00 2001
From: agorzyns <amber.gorzynski at amd.com>
Date: Wed, 4 Mar 2026 09:32:58 -0600
Subject: [PATCH] [WIP][Don't Merge] Fix formatting
---
llvm/lib/Target/AMDGPU/AMDGPUMarkLastScratchLoad.cpp | 1 +
llvm/lib/Target/AMDGPU/SILowerI1Copies.cpp | 1 +
llvm/lib/Target/AMDGPU/SILowerSGPRSpills.cpp | 5 +++--
llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp | 12 +++++++++---
llvm/lib/Target/SPIRV/SPIRVInstructionSelector.cpp | 8 ++++++--
5 files changed, 20 insertions(+), 7 deletions(-)
diff --git a/llvm/lib/Target/AMDGPU/AMDGPUMarkLastScratchLoad.cpp b/llvm/lib/Target/AMDGPU/AMDGPUMarkLastScratchLoad.cpp
index 9b6bb56c85d24..cda01e4228955 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUMarkLastScratchLoad.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUMarkLastScratchLoad.cpp
@@ -119,6 +119,7 @@ bool AMDGPUMarkLastScratchLoad::run(MachineFunction &MF) {
// instruction from the next slot.
if (!MISegmentEnd) {
SlotIndex NextSlot = Slots.getNextNonNullIndex(Segment.end);
+ abort();
MISegmentEnd = SI->getInstructionFromIndex(NextSlot);
}
diff --git a/llvm/lib/Target/AMDGPU/SILowerI1Copies.cpp b/llvm/lib/Target/AMDGPU/SILowerI1Copies.cpp
index 0b8c71a4a2453..871c5e7404881 100644
--- a/llvm/lib/Target/AMDGPU/SILowerI1Copies.cpp
+++ b/llvm/lib/Target/AMDGPU/SILowerI1Copies.cpp
@@ -644,6 +644,7 @@ bool Vreg1LoweringHelper::lowerCopiesToI1() {
unsigned FoundLoopLevel = LF.findLoop(PostDomBound);
if (FoundLoopLevel) {
SSAUpdater.Initialize(DstReg);
+ abort();
SSAUpdater.AddAvailableValue(&MBB, DstReg);
LF.addLoopEntries(FoundLoopLevel, SSAUpdater, *MRI, LaneMaskRegAttrs);
diff --git a/llvm/lib/Target/AMDGPU/SILowerSGPRSpills.cpp b/llvm/lib/Target/AMDGPU/SILowerSGPRSpills.cpp
index 196e551932659..045cf573dcdf4 100644
--- a/llvm/lib/Target/AMDGPU/SILowerSGPRSpills.cpp
+++ b/llvm/lib/Target/AMDGPU/SILowerSGPRSpills.cpp
@@ -340,9 +340,10 @@ void SILowerSGPRSpills::updateLaneVGPRDomInstr(
// Find the common dominator block between PrevInsertPt and the
// current spill.
DomMBB = MDT->findNearestCommonDominator(DomMBB, MBB);
- if (DomMBB == MBB)
+ if (DomMBB == MBB) {
I->second = InsertPt;
- else if (DomMBB != PrevInsertPt->getParent())
+ abort();
+ } else if (DomMBB != PrevInsertPt->getParent())
I->second = &(*DomMBB->getFirstTerminator());
}
}
diff --git a/llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp b/llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp
index 04add49846350..1d46ad2546e26 100644
--- a/llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp
@@ -759,8 +759,10 @@ Type *SPIRVEmitIntrinsics::getGEPType(GetElementPtrInst *Ref) {
// useful here
if (isNestedPointer(Ref->getSourceElementType())) {
Ty = Ref->getSourceElementType();
- for (Use &U : drop_begin(Ref->indices()))
+ for (Use &U : drop_begin(Ref->indices())) {
Ty = GetElementPtrInst::getTypeAtIndex(Ty, U.get());
+ abort();
+ }
} else {
Ty = Ref->getResultElementType();
}
@@ -1134,8 +1136,10 @@ void SPIRVEmitIntrinsics::deduceOperandElementTypeFunctionPointer(
IsNewFTy = true;
RetTy =
getTypedPointerWrapper(ElemTy, getPointerAddressSpace(CI->getType()));
- if (isTodoType(CI))
+ if (isTodoType(CI)) {
IsIncomplete = true;
+ abort();
+ }
} else {
IsIncomplete = true;
}
@@ -2658,8 +2662,10 @@ Type *SPIRVEmitIntrinsics::deduceFunParamElementType(
return KnownTy;
// try to deduce from the operand itself
Visited.clear();
- if (Type *Ty = deduceElementTypeHelper(OpArg, Visited, false))
+ if (Type *Ty = deduceElementTypeHelper(OpArg, Visited, false)) {
+ abort();
return Ty;
+ }
// search in actual parameter's users
for (User *OpU : OpArg->users()) {
Instruction *Inst = dyn_cast<Instruction>(OpU);
diff --git a/llvm/lib/Target/SPIRV/SPIRVInstructionSelector.cpp b/llvm/lib/Target/SPIRV/SPIRVInstructionSelector.cpp
index dd0830bb879f5..019df1489a10c 100644
--- a/llvm/lib/Target/SPIRV/SPIRVInstructionSelector.cpp
+++ b/llvm/lib/Target/SPIRV/SPIRVInstructionSelector.cpp
@@ -2356,10 +2356,14 @@ static unsigned getPtrCmpOpcode(unsigned Pred) {
static unsigned getBoolCmpOpcode(unsigned PredNum) {
auto Pred = static_cast<CmpInst::Predicate>(PredNum);
switch (Pred) {
- case CmpInst::ICMP_EQ:
+ case CmpInst::ICMP_EQ: {
+ abort();
return SPIRV::OpLogicalEqual;
- case CmpInst::ICMP_NE:
+ }
+ case CmpInst::ICMP_NE: {
+ abort();
return SPIRV::OpLogicalNotEqual;
+ }
default:
llvm_unreachable("Unknown predicate type for Bool comparison");
}
More information about the llvm-commits
mailing list