[PATCH] D53267: [CodeExtractor] Erase debug intrinsics in outlined thunks

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

In https://reviews.llvm.org/D53267#1265669, @vsk wrote:

> In https://reviews.llvm.org/D53267#1265641, @sebpop wrote:
>
> > Does this patch fix https://bugs.llvm.org/show_bug.cgi?id=22900 
> >  In which case you may want to add the testcases from that bug.
>
>
> It does. I'm not sure that there's extra value in including the attached test case -- it uses an old textual IR format that's hard to update, and reduces to something similar to the test in this patch.
>
> Fwiw I've tested this patch by building several internal frameworks and running Csmith over the weekend. I found no regressions. With Csmith, the testing method was to run `-Os -g -mllvm -hot-cold-split={true, false}` and check that the two CRCs were the same.


Thanks for the fix. I didn't see this patch before I updated that bug. This should fix my issue, which related to some llvm.dbg.value intrinsics in outlined code that weren't updated properly.


https://reviews.llvm.org/D53267