This is an archive of the discontinued LLVM Phabricator instance.

Differential D11794

Fix REPE, REPZ, REPNZ for intel syntax
ClosedPublic

Authored by myatsina on Aug 6 2015, 1:27 AM.

Details

Reviewers
mcrosier
Commits
rGd353c45eafd1: [X86] Fix REPE, REPZ, REPNZ for intel syntax
rL244584: [X86] Fix REPE, REPZ, REPNZ for intel syntax
Summary

Fix REPE, REPZ, REPNZ for intel syntax

Diff Detail

Repository
rL LLVM

Event Timeline

myatsina updated this revision to Diff 31355.Aug 6 2015, 1:27 AM
myatsina retitled this revision from to Fix REPE, REPZ, REPNZ for intel syntax.
myatsina updated this object.
myatsina added a reviewer: mcrosier.
myatsina set the repository for this revision to rL LLVM.
myatsina added a subscriber: llvm-commits.
myatsina updated this revision to Diff 31436.Aug 6 2015, 3:40 AM

removed "rep" case from the test

mcrosier edited edge metadata.Aug 6 2015, 6:04 AM

And by fix you mean we should be applying the mnemonic aliases for both the Intel and
AT&T dialect unconditionally?

test/MC/X86/intel-syntax.s
689 ↗(On Diff #31436)

Please put spaces between "//" and the "CHECK".

myatsina updated this revision to Diff 31614.Aug 9 2015, 1:40 AM
myatsina edited edge metadata.

Added space between '//' and 'CHECK:'

myatsina added a comment.Aug 9 2015, 1:43 AM

REPE, REPZ, REPNZ, REPNE should have mnemonics for Intel syntax as well.
Currently using these instructions causes compilation errors for Intel syntax.

mcrosier accepted this revision.Aug 10 2015, 6:33 AM
mcrosier edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Aug 10 2015, 6:33 AM
Closed by commit rL244584: [X86] Fix REPE, REPZ, REPNZ for intel syntax (authored by myatsina). · Explain WhyAug 11 2015, 4:28 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
llvm/
trunk/
lib/
Target/
X86/
6 lines
test/
MC/
X86/
13 lines

Diff 31789

llvm/trunk/lib/Target/X86/X86InstrInfo.td

Show First 20 Lines • Show All 2,534 Lines • ▼ Show 20 Lines
def : MnemonicAlias<"popa", "popal", "intel">, Requires<[In32BitMode]>; def : MnemonicAlias<"popa", "popal", "intel">, Requires<[In32BitMode]>;
def : MnemonicAlias<"pusha", "pushal", "intel">, Requires<[In32BitMode]>; def : MnemonicAlias<"pusha", "pushal", "intel">, Requires<[In32BitMode]>;
def : MnemonicAlias<"popa", "popaw", "att">, Requires<[In16BitMode]>; def : MnemonicAlias<"popa", "popaw", "att">, Requires<[In16BitMode]>;
def : MnemonicAlias<"pusha", "pushaw", "att">, Requires<[In16BitMode]>; def : MnemonicAlias<"pusha", "pushaw", "att">, Requires<[In16BitMode]>;
def : MnemonicAlias<"popa", "popal", "att">, Requires<[In32BitMode]>; def : MnemonicAlias<"popa", "popal", "att">, Requires<[In32BitMode]>;
def : MnemonicAlias<"pusha", "pushal", "att">, Requires<[In32BitMode]>; def : MnemonicAlias<"pusha", "pushal", "att">, Requires<[In32BitMode]>;
def : MnemonicAlias<"repe", "rep", "att">; def : MnemonicAlias<"repe", "rep">;
def : MnemonicAlias<"repz", "rep", "att">; def : MnemonicAlias<"repz", "rep">;
def : MnemonicAlias<"repnz", "repne", "att">; def : MnemonicAlias<"repnz", "repne">;
def : MnemonicAlias<"ret", "retw", "att">, Requires<[In16BitMode]>; def : MnemonicAlias<"ret", "retw", "att">, Requires<[In16BitMode]>;
def : MnemonicAlias<"ret", "retl", "att">, Requires<[In32BitMode]>; def : MnemonicAlias<"ret", "retl", "att">, Requires<[In32BitMode]>;
def : MnemonicAlias<"ret", "retq", "att">, Requires<[In64BitMode]>; def : MnemonicAlias<"ret", "retq", "att">, Requires<[In64BitMode]>;
def : MnemonicAlias<"salb", "shlb", "att">; def : MnemonicAlias<"salb", "shlb", "att">;
def : MnemonicAlias<"salw", "shlw", "att">; def : MnemonicAlias<"salw", "shlw", "att">;
def : MnemonicAlias<"sall", "shll", "att">; def : MnemonicAlias<"sall", "shll", "att">;
▲ Show 20 Lines • Show All 397 Lines • Show Last 20 Lines

llvm/trunk/test/MC/X86/intel-syntax.s

Show First 20 Lines • Show All 682 Lines • ▼ Show 20 Lines
// CHECK: outsl (%rsi), %dx // CHECK: outsl (%rsi), %dx
imul bx, 123 imul bx, 123
imul ebx, 123 imul ebx, 123
imul rbx, 123 imul rbx, 123
// CHECK: imulw $123, %bx // CHECK: imulw $123, %bx
// CHECK: imull $123, %ebx // CHECK: imull $123, %ebx
// CHECK: imulq $123, %rbx // CHECK: imulq $123, %rbx
repe cmpsb
repz cmpsb
repne cmpsb
repnz cmpsb
// CHECK: rep
// CHECK: cmpsb %es:(%rdi), (%rsi)
// CHECK: rep
// CHECK: cmpsb %es:(%rdi), (%rsi)
// CHECK: repne
// CHECK: cmpsb %es:(%rdi), (%rsi)
// CHECK: repne
// CHECK: cmpsb %es:(%rdi), (%rsi)