[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?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 5442 bytes
Desc: not available
More information about the llvm-commits