[PATCH] [X86] Fix pattern match for 32-to-64-bit zext in the presence of AssertSext

Nadav Rotem nrotem at apple.com
Wed Nov 5 11:38:57 PST 2014


Michael, 

Is there a way to reduce the size of the testcase? Do you need two basic blocks?  Can you add additional check lines to make sure that the ‘mov’ that you are matching is the one you are expecting?

Thanks,
Nadav

> On Nov 5, 2014, at 1:18 AM, Michael Kuperstein <michael.m.kuperstein at intel.com> wrote:
> 
> Uploaded diff with proper context, sorry about the first one.
> 
> http://reviews.llvm.org/D6128
> 
> Files:
>  lib/Target/X86/X86InstrCompiler.td
>  test/CodeGen/X86/2014-11-04-TruncZext.ll
> <D6128.15797.patch>





More information about the llvm-commits mailing list