[PATCH] D16915: Fix for bug 26465
Taewook Oh via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 5 14:53:25 PST 2016
twoh removed rL LLVM as the repository for this revision.
twoh updated this revision to Diff 47053.
twoh added a comment.
Move the comment inside the 'if' clause.
@spatel, I don't think we need a test case for "Replace ((sub A, B) != C) with (B != A-C) if A & C are constants." in this diff, because only the order of if statements is changed for that case. Totally agree on you that the function is too long. Maybe a separate diff for that?
http://reviews.llvm.org/D16915
Files:
lib/Transforms/InstCombine/InstCombineCompares.cpp
test/Transforms/InstCombine/prevent-cmp-merge.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16915.47053.patch
Type: text/x-patch
Size: 5442 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160205/20f80885/attachment.bin>
More information about the llvm-commits
mailing list