[PATCH] D13259: LLE 6/6: Add LoopLoadElimination pass
    hfinkel@anl.gov via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Nov  2 10:37:03 PST 2015
    
    
  
hfinkel added a comment.
LGTM too, except for:
I don't think we check anywhere to disqualify volatile/atomic loads/stores, and we need to do that (especially for volatiles, since you're changing the number of accesses, and I'd need to think more-carefully about atomics).
http://reviews.llvm.org/D13259
    
    
More information about the llvm-commits
mailing list