[PATCH] D12477: [LV] Don't bail to MiddleBlock if a runtime check fails, bail to ScalarPH instead

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 30 14:04:19 PDT 2015


mzolotukhin added a comment.

Hi James,

Won't we execute one iteration of scalar loop when trip-count is 0? IIUC, before the change we checked (0%VF==0) and went to MiddleBlock, where we checked it against 0, and then skipped the scalar loop. After the change we check (0%VF==0) and go straight to scalar pre-header. Thus we'll execute one iteration for this loop, which is incorrect. Do I miss something?


Repository:
  rL LLVM

http://reviews.llvm.org/D12477





More information about the llvm-commits mailing list