[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