[PATCH] D74517: [Local] Do not move around dbg.declares during replaceDbgDeclare

Evgenii Stepanov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 12 17:43:54 PST 2020


eugenis added a comment.

I think the point of moving dbg.declare was to make sure it is dominated by the new expression. Ex. in ASan we place it immediately after the base pointer definition. 
If you are sure this condition holds in all call sites, then LGTM.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74517/new/

https://reviews.llvm.org/D74517





More information about the llvm-commits mailing list