[PATCH] Break dependencies in large loops containing reductions	(LoopVectorize)
    hfinkel at anl.gov 
    hfinkel at anl.gov
       
    Thu Mar  5 18:30:44 PST 2015
    
    
  
================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:334
@@ -333,1 +333,3 @@
 
+  /// \brief Return true if interleaving is usually beneficial for the target.
+  /// This is typically the case when there is a substantial amount of
----------------
I don't like this description. I recommend just saying:
  /// \brief Don't restrict Interleaved unrolling to small loops. 
================
Comment at: include/llvm/Analysis/TargetTransformInfo.h:337
@@ +336,3 @@
+  /// registers, and hiding latency significantly improves performance.
+  bool enableAggressiveInterleaving() const;
+
----------------
Let's make this:
  bool enableAggressiveInterleaving(bool hasReductions) const;
because I'd like to enable this for all loops on the A2 (not just the ones with reductions)
http://reviews.llvm.org/D7514
EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
    
    
More information about the llvm-commits
mailing list