[x86] machine combiner reassociation: mark EFLAGS operand as 'dead'
authorSanjay Patel <spatel@rotateright.com>
Tue, 4 Aug 2015 15:21:56 +0000 (15:21 +0000)
committerSanjay Patel <spatel@rotateright.com>
Tue, 4 Aug 2015 15:21:56 +0000 (15:21 +0000)
commit91add497c683abdfa428686780d81124f3f07b53
treefec95556282597089906969a837b9a55608d9185
parentec5601a7f664c208face19af3ebc8e75f20a5ddb
[x86] machine combiner reassociation: mark EFLAGS operand as 'dead'

In the commentary for D11660, I wasn't sure if it was alright to create new
integer machine instructions without also creating the implicit EFLAGS operand.
From what I can see, the implicit operand is always created by the MachineInstrBuilder
based on the instruction type, so we don't have to do that explicitly. However, in
reviewing the debug output, I noticed that the operand was not marked as 'dead'.
The machine combiner should do that to preserve future optimization opportunities
that may be checking for that dead EFLAGS operand themselves.

Differential Revision: http://reviews.llvm.org/D11696

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@243990 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/X86/X86InstrInfo.cpp
test/CodeGen/X86/machine-combiner-int.ll