[PATCH] D50562: [sanitizer] Remove st(X) from the clobber list in 32-bit x86 atomics

[Kostya Kortchinsky via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rCRT339575: [sanitizer] Remove st(X) from the clobber list in 32-bit x86 atomics (authored by cryptoad, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D50562?vs=160114&id=160358#toc

Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D50562

Files:
  lib/sanitizer_common/sanitizer_atomic_clang_x86.h


Index: lib/sanitizer_common/sanitizer_atomic_clang_x86.h
===================================================================
--- lib/sanitizer_common/sanitizer_atomic_clang_x86.h
+++ lib/sanitizer_common/sanitizer_atomic_clang_x86.h
@@ -61,8 +61,7 @@
         "emms;"            // Empty mmx state/Reset FP regs
         : "=m" (v)
         : "m" (a->val_dont_use)
-        : // mark the FP stack and mmx registers as clobbered
-          "st", "st(1)", "st(2)", "st(3)", "st(4)", "st(5)", "st(6)", "st(7)",
+        : // mark the mmx registers as clobbered
 #ifdef __MMX__
           "mm0", "mm1", "mm2", "mm3", "mm4", "mm5", "mm6", "mm7",
 #endif  // #ifdef __MMX__
@@ -100,8 +99,7 @@
         "emms;"            // Empty mmx state/Reset FP regs
         : "=m" (a->val_dont_use)
         : "m" (v)
-        : // mark the FP stack and mmx registers as clobbered
-          "st", "st(1)", "st(2)", "st(3)", "st(4)", "st(5)", "st(6)", "st(7)",
+        : // mark the mmx registers as clobbered
 #ifdef __MMX__
           "mm0", "mm1", "mm2", "mm3", "mm4", "mm5", "mm6", "mm7",
 #endif  // #ifdef __MMX__


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D50562.160358.patch
Type: text/x-patch
Size: 1104 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180813/4f1d7ab2/attachment.bin>