[PATCH] D72631: [DSE] Eliminate stores at the end of the function.
    Alina Sbirlea via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Tue Jan 14 14:17:34 PST 2020
    
    
  
asbirlea added a comment.
Just a general comment here: DSE shouldn't necessarily make all optimizations possible. This one looks like it should be caught by DCE. If it's not, perhaps it would be good to improve DCE. Alternatively, if it's cheap enough, add a comment that it's still worth adding it to DSE.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72631/new/
https://reviews.llvm.org/D72631
    
    
More information about the llvm-commits
mailing list