Skip to content

Commit 4fe1a87

Browse files
authored
[X86][APX] Promote 8/16-bit LEA to 32-bit to avoid partial dependence (#161051)
1 parent 99f296d commit 4fe1a87

File tree

4 files changed

+19
-17
lines changed

4 files changed

+19
-17
lines changed

llvm/lib/Target/X86/X86InstrArithmetic.td

Lines changed: 11 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -25,18 +25,12 @@ let SchedRW = [WriteLEA] in {
2525
[(set GR32:$dst, lea32addr:$src)]>,
2626
OpSize32, Requires<[Not64BitMode]>;
2727

28-
let Predicates = [HasNDD], isCodeGenOnly = 1 in {
29-
def LEA64_8r : I<0x8D, MRMSrcMem, (outs GR8:$dst), (ins lea64_8mem:$src),
30-
"lea{b}\t{$src|$dst}, {$dst|$src}",
31-
[(set GR8:$dst, lea64_iaddr:$src)]>,
32-
OpSize16,
33-
Requires<[In64BitMode]>;
34-
35-
def LEA64_16r : I<0x8D, MRMSrcMem, (outs GR16:$dst), (ins lea64_16mem:$src),
36-
"lea{w}\t{$src|$dst}, {$dst|$src}",
37-
[(set GR16:$dst, lea64_iaddr:$src)]>,
38-
OpSize16,
39-
Requires<[In64BitMode]>;
28+
let isCodeGenOnly = 1 in {
29+
def LEA64_8r : I<0x8D, MRMSrcMem, (outs GR32:$dst), (ins lea64_8mem:$src),
30+
"lea{l}\t{$src|$dst}, {$dst|$src}", []>, OpSize32;
31+
32+
def LEA64_16r : I<0x8D, MRMSrcMem, (outs GR32:$dst), (ins lea64_16mem:$src),
33+
"lea{l}\t{$src|$dst}, {$dst|$src}", []>, OpSize32;
4034
}
4135

4236
def LEA64_32r : I<0x8D, MRMSrcMem, (outs GR32:$dst), (ins lea64_32mem:$src),
@@ -51,6 +45,11 @@ let SchedRW = [WriteLEA] in {
5145
[(set GR64:$dst, lea64addr:$src)]>;
5246
} // SchedRW
5347

48+
let Predicates = [HasNDD] in {
49+
def : Pat<(i8 lea64_iaddr:$src), (EXTRACT_SUBREG (LEA64_8r lea64_8mem:$src), sub_8bit)>;
50+
def : Pat<(i16 lea64_iaddr:$src), (EXTRACT_SUBREG (LEA64_16r lea64_16mem:$src), sub_16bit)>;
51+
}
52+
5453
// Pseudo instruction for lea that prevent optimizer from eliminating
5554
// the instruction.
5655
let SchedRW = [WriteLEA], isPseudo = true, hasSideEffects = 1 in {

llvm/test/CodeGen/X86/apx/ndd-neg-addr-index.ll

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ define void @neg_8bit_1(i1 %cmp) {
88
; NDD-NEXT: andb $1, %dil, %al # encoding: [0x62,0xf4,0x7c,0x18,0x80,0xe7,0x01]
99
; NDD-NEXT: movzbl 0, %ecx # encoding: [0x0f,0xb6,0x0c,0x25,0x00,0x00,0x00,0x00]
1010
; NDD-NEXT: negb %al, %al # encoding: [0x62,0xf4,0x7c,0x18,0xf6,0xd8]
11-
; NDD-NEXT: leab 2(%rcx,%rax), %al # encoding: [0x66,0x8d,0x44,0x01,0x02]
11+
; NDD-NEXT: leal 2(%rcx,%rax), %eax # encoding: [0x8d,0x44,0x01,0x02]
1212
; NDD-NEXT: movb %al, 0 # encoding: [0x88,0x04,0x25,0x00,0x00,0x00,0x00]
1313
; NDD-NEXT: retq # encoding: [0xc3]
1414
entry:
@@ -25,7 +25,8 @@ define void @neg_8bit_2(i8 %int8) {
2525
; NDD-NEXT: # kill: def $edi killed $edi def $rdi
2626
; NDD-NEXT: addb %dil, %dil, %al # encoding: [0x62,0xf4,0x7c,0x18,0x00,0xff]
2727
; NDD-NEXT: negb %al, %al # encoding: [0x62,0xf4,0x7c,0x18,0xf6,0xd8]
28-
; NDD-NEXT: leab 1(%rdi,%rax), %al # encoding: [0x66,0x8d,0x44,0x07,0x01]
28+
; NDD-NEXT: leal 1(%rdi,%rax), %eax # encoding: [0x8d,0x44,0x07,0x01]
29+
; NDD-NEXT: # kill: def $al killed $al killed $eax
2930
; NDD-NEXT: mulb %dil # encoding: [0x40,0xf6,0xe7]
3031
; NDD-NEXT: testb %al, %al # encoding: [0x84,0xc0]
3132
; NDD-NEXT: retq # encoding: [0xc3]
@@ -55,7 +56,7 @@ define i32 @neg_16bit(i16 %0) {
5556
; NDD-NEXT: cmovsl %ecx, %eax # EVEX TO LEGACY Compression encoding: [0x0f,0x48,0xc1]
5657
; NDD-NEXT: andw $-256, %ax # EVEX TO LEGACY Compression encoding: [0x66,0x25,0x00,0xff]
5758
; NDD-NEXT: negw %ax, %ax # encoding: [0x62,0xf4,0x7d,0x18,0xf7,0xd8]
58-
; NDD-NEXT: leaw 1(%rdi,%rax), %ax # encoding: [0x66,0x8d,0x44,0x07,0x01]
59+
; NDD-NEXT: leal 1(%rdi,%rax), %eax # encoding: [0x8d,0x44,0x07,0x01]
5960
; NDD-NEXT: movzwl %ax, %eax # encoding: [0x0f,0xb7,0xc0]
6061
; NDD-NEXT: movq %rax, 0 # encoding: [0x48,0x89,0x04,0x25,0x00,0x00,0x00,0x00]
6162
; NDD-NEXT: xorl %eax, %eax # encoding: [0x31,0xc0]

llvm/test/CodeGen/X86/lea-16bit.ll

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ define i16 @lea16bit(i16 %in) {
1313
; NDD-LABEL: lea16bit:
1414
; NDD: # %bb.0:
1515
; NDD-NEXT: # kill: def $edi killed $edi def $rdi
16-
; NDD-NEXT: leaw 1(%rdi,%rdi), %ax
16+
; NDD-NEXT: leal 1(%rdi,%rdi), %eax
17+
; NDD-NEXT: # kill: def $ax killed $ax killed $eax
1718
; NDD-NEXT: retq
1819
%shl = shl i16 %in, 1
1920
%or = or i16 %shl, 1

llvm/test/CodeGen/X86/lea-8bit.ll

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,8 @@ define i8 @lea8bit(i8 %in) {
1414
; NDD-LABEL: lea8bit:
1515
; NDD: # %bb.0:
1616
; NDD-NEXT: # kill: def $edi killed $edi def $rdi
17-
; NDD-NEXT: leab 1(%rdi,%rdi), %al
17+
; NDD-NEXT: leal 1(%rdi,%rdi), %eax
18+
; NDD-NEXT: # kill: def $al killed $al killed $eax
1819
; NDD-NEXT: retq
1920
%shl = shl i8 %in, 1
2021
%or = or i8 %shl, 1

0 commit comments

Comments
 (0)