[PATCH] D32892: Handle a COPY with undef source operand in LowerCopy().

Wed May 10 01:22:28 PDT 2017

jonpa updated this revision to Diff 98413.
jonpa added a comment.

> This is barely one workday old (in my timezone).

Sorry - got carried away since I was working a little over the weekend :-)

> The code changes look good to me, but this would better be tested with a .mir test.

Thanks. I changed the test.

Btw, I wonder what the best way to test that the machine-verifier passes? Currently I just do a CHECK for some expected string. Is this what "not llc" might be used for?

https://reviews.llvm.org/D32892

Files:
  lib/CodeGen/ExpandPostRAPseudos.cpp
  test/CodeGen/SystemZ/lower-copy-undef-src.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32892.98413.patch
Type: text/x-patch
Size: 8311 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170510/79837f3f/attachment.bin>