diff options
| author | Craig Topper <craig.topper@intel.com> | 2019-02-03 07:53:39 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2019-02-03 07:53:39 +0000 |
| commit | 5a570dd4374daaa1ea3110c507d852c68581c4c4 (patch) | |
| tree | fa97cdca480d5f7fdd3065bf6ae6a43d6c12874f /llvm/test/CodeGen/X86/fp128-cast.ll | |
| parent | 950ca192f6f606beb44dcbb0b9f203eed69dadcb (diff) | |
| download | bcm5719-llvm-5a570dd4374daaa1ea3110c507d852c68581c4c4.tar.gz bcm5719-llvm-5a570dd4374daaa1ea3110c507d852c68581c4c4.zip | |
[X86] Print %st(0) as %st to match what gcc inline asm uses as the clobber name to make MS inline asm work correctly
Summary:
When calculating clobbers for MS style inline assembly we fail if the asm clobbers stack top because we print st(0) and try to pass it through the gcc register name check. This was found with when I attempted to make a emms/femms clobber all ST registers. If you use emms/femms in MS inline asm we would try to use st(0) as the clobber name but clang would think that wasn't a valid clobber name.
This also matches what objdump disassembly prints. It's also what is printed by gcc -S.
Reviewers: RKSimon, rnk, efriedma, spatel, andreadb, lebedev.ri
Reviewed By: rnk
Subscribers: eraman, gbedwell, lebedev.ri, llvm-commits
Differential Revision: https://reviews.llvm.org/D57621
llvm-svn: 352985
Diffstat (limited to 'llvm/test/CodeGen/X86/fp128-cast.ll')
| -rw-r--r-- | llvm/test/CodeGen/X86/fp128-cast.ll | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/test/CodeGen/X86/fp128-cast.ll b/llvm/test/CodeGen/X86/fp128-cast.ll index 71b9c3f7f7b..4dc119391b8 100644 --- a/llvm/test/CodeGen/X86/fp128-cast.ll +++ b/llvm/test/CodeGen/X86/fp128-cast.ll @@ -900,7 +900,7 @@ define fp128 @TestTruncCopysign(fp128 %x, i32 %n) nounwind { ; X32-NEXT: fstp %st(1) ; X32-NEXT: fldz ; X32-NEXT: .LBB17_3: # %if.then -; X32-NEXT: fstp %st(0) +; X32-NEXT: fstp %st ; X32-NEXT: subl $16, %esp ; X32-NEXT: leal {{[0-9]+}}(%esp), %eax ; X32-NEXT: movl %eax, (%esp) |
