[PATCH] D20649: [X86] Add a pattern that uses GR16_ABCD rather than GR32_ABCD to avoid falsely marking whole 32 bit register as live.

[Sanjay Patel via llvm-commits]

spatel added reviewers: MatzeB, mbodart, qcolombet.
spatel added a comment.

[cc'ing Matthias, Mitch, Quentin ; a related test suggestion was made in http://reviews.llvm.org/D20456]

The code change itself looks fine to me, but I wonder if there's a preference for the test case to use "-stop-after" instead (similar to how MIR tests are written):

; RUN: llc -march=x86 -stop-after=expand-isel-pseudos <%s 2>&1 | FileCheck %s

And then either CHECK for the exact expected sequence of machine instructions or CHECK-NOT the instructions that are generated without the patch:

  %5 = INSERT_SUBREG %6, %0, 3
  %7 = COPY %5


http://reviews.llvm.org/D20649