[PATCH] D41944: [LLVM][IR][LIT] support of 'no-overflow' flag for sdiv\udiv instructions

Mohammed Agabaria via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 25 07:37:31 PST 2018


magabari marked 15 inline comments as done.
magabari added a comment.

fixed



================
Comment at: lib/CodeGen/CodeGenPrepare.cpp:647
+  Value *VResult = UndefVal;
+  Value *PrevPhi = UndefVal;
+  unsigned VectorWidth = I.getType()->getVectorNumElements();
----------------
craig.topper wrote:
> Is the starting value of PrevPhi ever expected to be used? Does it need to be undef?
PrevPhi will be used at the first merge "undef" i think you meant VResult which can be uninitialized.


https://reviews.llvm.org/D41944





More information about the llvm-commits mailing list