[PATCH] D33603: [SimplifyInst] Push commuted op checks for and/or of icmp further down to avoid duplicate work

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 26 12:42:17 PDT 2017


craig.topper created this revision.

Previously, we called simplifyPossiblyCastedAndOrOfICmps twice with the operands commuted, but the call to simplifyAndOrOfICmpsWithConstants further down already handles commuting and doesn't need to be called both ways.

This patch pushes double calls further down to just the individual routines that need to be called twice.


https://reviews.llvm.org/D33603

Files:
  lib/Analysis/InstructionSimplify.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33603.100453.patch
Type: text/x-patch
Size: 4452 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170526/2083731d/attachment.bin>


More information about the llvm-commits mailing list