[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]

Summary

The backend used to print the x87 FPSW register as 'fpsw', but gcc inline asm uses 'fpsr'. After D57641, the backend now uses 'fpsr' to match. So we need to update the test and we can remove the code that excluded the 'fpsw' string.

[rnk]

test/CodeGen/ms-inline-asm.c

574 ↗

(On Diff #184917)

Two fpsr? If we manually add that as an extra constraint, maybe we should remove that logic in this change.

[craig.topper]

test/CodeGen/ms-inline-asm.c

574 ↗

(On Diff #184917)

We slap "~{dirflag},~{fpsr},~{flags}" blindly onto both gcc and MS inline assembly. There are tests cases in this file with two ~{flags} as well.

[rnk]

lib/Parse/ParseStmtAsm.cpp

637–639 ↗

(On Diff #184917)

In light of this comment about "clang always adding fpsr anyway" I'd rather keep the fpsr filter here.

[craig.topper]

Keep the filter on fpsw, but change the string to fpsr

[rnk]

Thanks!

cfe/trunk/lib/Parse/ParseStmtAsm.cpp

StmtResult Parser::ParseMicrosoftAsmStatement(SourceLocation AsmLoc) {
   if (Parser->parseMSInlineAsm(AsmLoc.getPtrEncoding(), AsmStringIR, NumOutputs,
                                NumInputs, OpExprs, Constraints, Clobbers,
                                MII.get(), IP.get(), Callback))
     return StmtError();
 
   // 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.
   ClobberRefs.insert(ClobberRefs.end(), Clobbers.begin(), Clobbers.end());
 
   // Recast the void pointers and build the vector of constraint StringRefs.
   unsigned NumExprs = NumOutputs + NumInputs;
   ConstraintRefs.resize(NumExprs);