[llvm] [DebugInfo][JumpThreading] Fix missing debug location updates (PR #91581)
Shan Huang via llvm-commits
llvm-commits at lists.llvm.org
Thu May 9 06:16:22 PDT 2024
================
@@ -1278,9 +1278,12 @@ bool JumpThreadingPass::simplifyPartiallyRedundantLoad(LoadInst *LoadI) {
// only happen in dead loops.
if (AvailableVal == LoadI)
AvailableVal = PoisonValue::get(LoadI->getType());
- if (AvailableVal->getType() != LoadI->getType())
+ if (AvailableVal->getType() != LoadI->getType()) {
AvailableVal = CastInst::CreateBitOrPointerCast(
AvailableVal, LoadI->getType(), "", LoadI->getIterator());
+ if (Instruction *CI = dyn_cast<Instruction>(AvailableVal))
+ CI->setDebugLoc(LoadI->getDebugLoc());
----------------
Apochens wrote:
Yes. For each case like this, I would first search for existing tests by sticking an `assert(false)`. If any test is found, I modify that test to obtain a test for the patched site. For this site, I have not found any existing test, which can trigger it.
https://github.com/llvm/llvm-project/pull/91581
More information about the llvm-commits
mailing list