[PATCH] D57642: [X86] Update clobber list in a test after D57641. Remove filter for 'fpsw' in MS inline asm clobber list generation since the backend now uses 'fpsr'.
Craig Topper via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 4 17:27:27 PST 2019
craig.topper updated this revision to Diff 185201.
craig.topper added a comment.
Keep the filter on fpsw, but change the string to fpsr
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D57642/new/
https://reviews.llvm.org/D57642
Files:
lib/Parse/ParseStmtAsm.cpp
Index: lib/Parse/ParseStmtAsm.cpp
===================================================================
--- lib/Parse/ParseStmtAsm.cpp
+++ lib/Parse/ParseStmtAsm.cpp
@@ -636,7 +636,7 @@
// Filter out "fpsw" and "mxcsr". They aren't valid GCC asm clobber
// constraints. Clang always adds fpsr to the clobber list anyway.
llvm::erase_if(Clobbers, [](const std::string &C) {
- return C == "fpsw" || C == "mxcsr";
+ return C == "fpsr" || C == "mxcsr";
});
// Build the vector of clobber StringRefs.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D57642.185201.patch
Type: text/x-patch
Size: 517 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190205/b76e45f6/attachment.bin>
More information about the cfe-commits
mailing list