[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