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

Michael Zolotukhin mzolotukhin at apple.com
Tue Apr 28 16:56:14 PDT 2015


Hi aschwaighofer, nadav, hfinkel, anemet,

Currently this code is very hard to manage:
 - variables defined in the beginning of the function might be used several
   screens later
 - Code for adding checks isn't uniform enough (some use Instruction::Create,
   others use IRBuilder)
 - Code for adding one check is tangled with code for adding next check, because
   we split basic block only at that time.
I tried to fix this issues, and hopefully that improves the code readability.

http://reviews.llvm.org/D9332

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/induction.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D9332.24586.patch
Type: text/x-patch
Size: 11769 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150428/4fd0445f/attachment.bin>


More information about the llvm-commits mailing list