[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