[AMDGPU] expand-fp: always report modifications (#163153)

frederik-h · web-flow · commit 8cc862ce3bd5 · 2025-10-13T15:21:02.000Z
The last change to the pass in PR #158588 lost the assignment to the "Modified" variable for one of the pass optimizations. Add it back. This fixes the test failure in `CodeGen/AMDGPU/itofp.i128.bf.ll` (in a `LLVM_ENABLE_EXPENSIVE_CHECKS=ON` build).
diff --git a/llvm/lib/CodeGen/ExpandFp.cpp b/llvm/lib/CodeGen/ExpandFp.cpp
@@ -1036,6 +1036,7 @@ static bool runImpl(Function &F, const TargetLowering &TLI,
         continue;
 
       addToWorklist(I, Worklist);
+      Modified = true;
       break;
     }
     default:

Original file line number	Diff line number	Diff line change
`@@ -1036,6 +1036,7 @@ static bool runImpl(Function &F, const TargetLowering &TLI,`
`1036`	`1036`	`continue;`
`1037`	`1037`
`1038`	`1038`	`addToWorklist(I, Worklist);`
	`1039`	`+ Modified = true;`
`1039`	`1040`	`break;`
`1040`	`1041`	`}`
`1041`	`1042`	`default:`