[cfe-commits] r161988 - /cfe/trunk/lib/Sema/SemaStmt.cpp
Chad Rosier
mcrosier at apple.com
Wed Aug 15 14:08:52 PDT 2012
Author: mcrosier
Date: Wed Aug 15 16:08:52 2012
New Revision: 161988
URL: http://llvm.org/viewvc/llvm-project?rev=161988&view=rev
Log:
[ms-inline asm] Address a FIXME by computing the number of asm statements when
building the AsmString.
Modified:
cfe/trunk/lib/Sema/SemaStmt.cpp
Modified: cfe/trunk/lib/Sema/SemaStmt.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaStmt.cpp?rev=161988&r1=161987&r2=161988&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaStmt.cpp (original)
+++ cfe/trunk/lib/Sema/SemaStmt.cpp Wed Aug 15 16:08:52 2012
@@ -2840,17 +2840,20 @@
// Build the unmodified MSAsmString.
static std::string buildMSAsmString(Sema &SemaRef,
- ArrayRef<Token> AsmToks) {
+ ArrayRef<Token> AsmToks,
+ unsigned &NumAsmStrings) {
assert (!AsmToks.empty() && "Didn't expect an empty AsmToks!");
SmallString<512> Asm;
SmallString<512> TokenBuf;
TokenBuf.resize(512);
+ NumAsmStrings = 0;
for (unsigned i = 0, e = AsmToks.size(); i < e; ++i) {
bool isNewAsm = i == 0 || AsmToks[i].isAtStartOfLine() ||
AsmToks[i].is(tok::kw_asm);
if (isNewAsm) {
+ ++NumAsmStrings;
if (i)
Asm += '\n';
if (AsmToks[i].is(tok::kw_asm)) {
@@ -2889,18 +2892,12 @@
return Owned(NS);
}
- std::string AsmString = buildMSAsmString(*this, AsmToks);
+ unsigned NumAsmStrings;
+ std::string AsmString = buildMSAsmString(*this, AsmToks, NumAsmStrings);
bool IsSimple;
std::vector<std::string> PatchedAsmStrings;
-
- // FIXME: Count this while parsing.
- unsigned NumAsmStrings = 0;
- for (unsigned i = 0, e = AsmToks.size(); i != e; ++i)
- if (i == 0 || AsmToks[i].isAtStartOfLine() || AsmToks[i].is(tok::kw_asm))
- ++NumAsmStrings;
-
- PatchedAsmStrings.resize(NumAsmStrings ? NumAsmStrings : 1);
+ PatchedAsmStrings.resize(NumAsmStrings);
// Rewrite operands to appease the AsmParser.
patchMSAsmStrings(*this, IsSimple, AsmLoc, AsmToks, Context.getTargetInfo(),
More information about the cfe-commits
mailing list