[PATCH] D43530: [X86] Make the subus special case in LowerVSETCC self contained

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 20 14:26:43 PST 2018


craig.topper created this revision.
craig.topper added reviewers: spatel, RKSimon.

Previously this code overrode the flags and opcode used by the later code in LowerVSETCC. This makes the code difficult to read and follow.

This patch moves all the SUBUS code into its own function and makes it responsible for creating its own SDNodes on success.


https://reviews.llvm.org/D43530

Files:
  lib/Target/X86/X86ISelLowering.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D43530.135147.patch
Type: text/x-patch
Size: 4188 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180220/f2a9c6fe/attachment.bin>


More information about the llvm-commits mailing list