[PATCH] D56848: [LCSSA] Skip blocks in sub-loops when scanning for uses.
    Eli Friedman via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Jan 17 12:40:16 PST 2019
    
    
  
efriedma added inline comments.
================
Comment at: llvm/lib/Transforms/Utils/LCSSA.cpp:328
   for (BasicBlock *BB : BlocksDominatingExits) {
+    // Skip blocks that are part of any sub-loops, as we already scanned them
+    // previously.
----------------
The relevant property here is that the sub-loops are already in LCSSA form, right?  The documentation for formLCSSA doesn't actually say that explicitly... granted, it's probably true in the cases we care about.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56848/new/
https://reviews.llvm.org/D56848
    
    
More information about the llvm-commits
mailing list