[PATCH] X86: Correct Defs for instructions that imp-def both an A-register and EFLAGS.

Ahmed Bougacha ahmed.bougacha at gmail.com
Wed May 29 11:59:33 PDT 2013


Hi Jakob, all,

Title says all: this patch corrects a problem where x86 instructions
that implicitly define both an A-register (RAX, EAX, ..) and EFLAGS
were declared as only defining EFLAGS, because the outer
  let Defs = [EFLAGS]
in the various multiclasses overrides the
  let Defs = [areg]
in BinOpAI.

Add EFLAGS to BinOpAIs, and move all instances outside "let Defs".

Thanks,

-- Ahmed Bougacha
-------------- next part --------------
A non-text attachment was scrubbed...
Name: x86-binopa-eflags-impdef.diff
Type: application/octet-stream
Size: 8210 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130529/41e6472f/attachment.obj>


More information about the llvm-commits mailing list