[PATCH] Refactor code for adding runtime checks in vectorizer.

Michael Zolotukhin mzolotukhin at apple.com
Wed May 27 10:48:41 PDT 2015


Hi Hal,

I can't commit 's/BypassBlock/VectorPreHeader/' since the original code already has `VectorPH`, and having both `VectorPH` and `VectorPreHeader` simultaneously would be even more confusing. I'll try to remove as many other changes as possible though, and keep only the changes needed for this rename in a separate patch.

Thanks,
Michael


================
Comment at: test/Transforms/LoopVectorize/induction.ll:116
@@ -115,3 +115,1 @@
 ; CHECK-LABEL: max_i32_backedgetaken
-; CHECK:  %backedge.overflow = icmp eq i32 -1, -1
-; CHECK:  br i1 %backedge.overflow, label %scalar.ph, label %overflow.checked
----------------
hfinkel wrote:
> You've removed the check for the icmp. Why?
I started using IRBuilder, and it folded this expression.

http://reviews.llvm.org/D9332

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list