[llvm] [ReachingDefAnalysis] Fix management of MBBFrameObjsReachingDefs (PR #124943)
Michael Maitland via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 29 10:32:32 PST 2025
michaelmaitland wrote:
I had some trouble writing a test since we were sort of getting lucky. What I mean by that is `lookup` was returning a empty list of defs as a default value which was exactly what would happen when there is no reaching defs. I think that's why it did not impact the tests negatively. But this is definitely more robust. That makes that part of the patch "NFC-ish" i think, although I don't want to call it NFC because it may be fixing a tricky to find bug.
WRT to the Frame2InstrIdx bug, I would be surprised if that was NFC. I will try and write a test for it.
https://github.com/llvm/llvm-project/pull/124943
More information about the llvm-commits
mailing list