[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