[PATCH] D36303: [Polly][PM] Make the new-pm passes behave more like the legacy passes
    Tobias Grosser via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Aug  4 04:16:46 PDT 2017
    
    
  
grosser accepted this revision.
grosser added a comment.
This revision is now accepted and ready to land.
LGTM.
We may want to discuss if a better path ordering is possible, but I would prefer to have  (close-to) full test coverage before improving this.
================
Comment at: lib/Analysis/ScopInfo.cpp:5293
   auto &SI = FAM.getResult<ScopInfoAnalysis>(F);
-  for (auto &It : SI) {
+  for (auto &It : reverse(SI)) {
     if (It.second)
----------------
Maybe add a comment why you are doing this.
https://reviews.llvm.org/D36303
    
    
More information about the llvm-commits
mailing list