[llvm] fb99424 - [SystemZ][z/OS] Introduce initial support for GOFF asm parser
Anirudh Prasad via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 1 07:29:20 PDT 2021
Author: Anirudh Prasad
Date: 2021-10-01T10:29:14-04:00
New Revision: fb99424a6f65b5a68902a59e0b84d72607f8884d
URL: https://github.com/llvm/llvm-project/commit/fb99424a6f65b5a68902a59e0b84d72607f8884d
DIFF: https://github.com/llvm/llvm-project/commit/fb99424a6f65b5a68902a59e0b84d72607f8884d.diff
LOG: [SystemZ][z/OS] Introduce initial support for GOFF asm parser
- Introduce a skeleton outline for the GOFFAsmParser
- Before instantiating AsmParser/HLASMAsmParser, target specific asm parsers are attempted to be initialized first before proceeding. If it doesn't exist for a particular file type, we report a fatal error.
- This patch allows to properly instantiate the HLASMAsmParser on z/OS, and ensures we can write lit tests and unit tests which will involve the instantiation of asm parsers, without an assert / fatal error.
Reviewed By: uweigand, Kai
Differential Revision: https://reviews.llvm.org/D110730
Added:
llvm/lib/MC/MCParser/GOFFAsmParser.cpp
Modified:
llvm/lib/MC/MCParser/AsmParser.cpp
llvm/lib/MC/MCParser/CMakeLists.txt
llvm/unittests/MC/SystemZ/SystemZAsmLexerTest.cpp
Removed:
################################################################################
diff --git a/llvm/lib/MC/MCParser/AsmParser.cpp b/llvm/lib/MC/MCParser/AsmParser.cpp
index d5f01d6a49f42..ed9f2066dc209 100644
--- a/llvm/lib/MC/MCParser/AsmParser.cpp
+++ b/llvm/lib/MC/MCParser/AsmParser.cpp
@@ -749,6 +749,7 @@ namespace llvm {
extern MCAsmParserExtension *createDarwinAsmParser();
extern MCAsmParserExtension *createELFAsmParser();
extern MCAsmParserExtension *createCOFFAsmParser();
+extern MCAsmParserExtension *createGOFFAsmParser();
extern MCAsmParserExtension *createXCOFFAsmParser();
extern MCAsmParserExtension *createWasmAsmParser();
@@ -783,7 +784,8 @@ AsmParser::AsmParser(SourceMgr &SM, MCContext &Ctx, MCStreamer &Out,
PlatformParser.reset(createELFAsmParser());
break;
case MCContext::IsGOFF:
- report_fatal_error("GOFFAsmParser support not implemented yet");
+ PlatformParser.reset(createGOFFAsmParser());
+ break;
case MCContext::IsWasm:
PlatformParser.reset(createWasmAsmParser());
break;
diff --git a/llvm/lib/MC/MCParser/CMakeLists.txt b/llvm/lib/MC/MCParser/CMakeLists.txt
index f70787ad9d20e..81f6a730bc11d 100644
--- a/llvm/lib/MC/MCParser/CMakeLists.txt
+++ b/llvm/lib/MC/MCParser/CMakeLists.txt
@@ -3,6 +3,7 @@ add_llvm_component_library(LLVMMCParser
AsmParser.cpp
COFFAsmParser.cpp
COFFMasmParser.cpp
+ GOFFAsmParser.cpp
DarwinAsmParser.cpp
ELFAsmParser.cpp
MCAsmLexer.cpp
diff --git a/llvm/lib/MC/MCParser/GOFFAsmParser.cpp b/llvm/lib/MC/MCParser/GOFFAsmParser.cpp
new file mode 100644
index 0000000000000..c2a7eaee8029f
--- /dev/null
+++ b/llvm/lib/MC/MCParser/GOFFAsmParser.cpp
@@ -0,0 +1,48 @@
+//===- GOFFAsmParser.cpp - GOFF Assembly Parser ---------------------------===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#include "llvm/ADT/StringSwitch.h"
+#include "llvm/ADT/Twine.h"
+#include "llvm/MC/MCAsmInfo.h"
+#include "llvm/MC/MCContext.h"
+#include "llvm/MC/MCExpr.h"
+#include "llvm/MC/MCParser/MCAsmLexer.h"
+#include "llvm/MC/MCParser/MCAsmParserExtension.h"
+#include "llvm/MC/MCSectionGOFF.h"
+#include "llvm/MC/MCStreamer.h"
+#include "llvm/MC/MCSymbolGOFF.h"
+
+using namespace llvm;
+
+namespace {
+
+class GOFFAsmParser : public MCAsmParserExtension {
+ template <bool (GOFFAsmParser::*HandlerMethod)(StringRef, SMLoc)>
+ void addDirectiveHandler(StringRef Directive) {
+ MCAsmParser::ExtensionDirectiveHandler Handler =
+ std::make_pair(this, HandleDirective<GOFFAsmParser, HandlerMethod>);
+
+ getParser().addDirectiveHandler(Directive, Handler);
+ }
+
+public:
+ GOFFAsmParser() {}
+
+ void Initialize(MCAsmParser &Parser) override {
+ // Call the base implementation.
+ this->MCAsmParserExtension::Initialize(Parser);
+ }
+};
+
+} // namespace
+
+namespace llvm {
+
+MCAsmParserExtension *createGOFFAsmParser() { return new GOFFAsmParser; }
+
+} // namespace llvm
diff --git a/llvm/unittests/MC/SystemZ/SystemZAsmLexerTest.cpp b/llvm/unittests/MC/SystemZ/SystemZAsmLexerTest.cpp
index dd95b047d5bb8..1682570789fae 100644
--- a/llvm/unittests/MC/SystemZ/SystemZAsmLexerTest.cpp
+++ b/llvm/unittests/MC/SystemZ/SystemZAsmLexerTest.cpp
@@ -22,36 +22,6 @@ using namespace llvm;
namespace {
-// Come up with our hacked version of MCAsmInfo.
-// This hacked version derives from the main MCAsmInfo instance.
-// Here, we're free to override whatever we want, without polluting
-// the main MCAsmInfo interface.
-class MockedUpMCAsmInfo : public MCAsmInfo {
-public:
- void setRestrictCommentStringToStartOfStatement(bool Value) {
- RestrictCommentStringToStartOfStatement = Value;
- }
- void setCommentString(StringRef Value) { CommentString = Value; }
- void setAllowAdditionalComments(bool Value) {
- AllowAdditionalComments = Value;
- }
- void setAllowQuestionAtStartOfIdentifier(bool Value) {
- AllowQuestionAtStartOfIdentifier = Value;
- }
- void setAllowAtAtStartOfIdentifier(bool Value) {
- AllowAtAtStartOfIdentifier = Value;
- }
- void setAllowDollarAtStartOfIdentifier(bool Value) {
- AllowDollarAtStartOfIdentifier = Value;
- }
- void setAllowHashAtStartOfIdentifier(bool Value) {
- AllowHashAtStartOfIdentifier = Value;
- }
- void setAllowDotIsPC(bool Value) { DotIsPC = Value; }
- void setAssemblerDialect(unsigned Value) { AssemblerDialect = Value; }
- void setEmitLabelsInUpperCase(bool Value) { EmitLabelsInUpperCase = Value; }
-};
-
// Setup a testing class that the GTest framework can call.
class SystemZAsmLexerTest : public ::testing::Test {
protected:
@@ -62,7 +32,7 @@ class SystemZAsmLexerTest : public ::testing::Test {
}
std::unique_ptr<MCRegisterInfo> MRI;
- std::unique_ptr<MockedUpMCAsmInfo> MUPMAI;
+ std::unique_ptr<MCAsmInfo> MAI;
std::unique_ptr<const MCInstrInfo> MII;
std::unique_ptr<MCObjectFileInfo> MOFI;
std::unique_ptr<MCStreamer> Str;
@@ -78,10 +48,12 @@ class SystemZAsmLexerTest : public ::testing::Test {
const MCTargetOptions MCOptions;
- SystemZAsmLexerTest() {
+ SystemZAsmLexerTest() = delete;
+
+ SystemZAsmLexerTest(std::string SystemZTriple) {
// We will use the SystemZ triple, because of missing
// Object File and Streamer support for the z/OS target.
- TripleName = "s390x-ibm-linux";
+ TripleName = SystemZTriple;
Triple = llvm::Triple(TripleName);
std::string Error;
@@ -97,16 +69,8 @@ class SystemZAsmLexerTest : public ::testing::Test {
STI.reset(TheTarget->createMCSubtargetInfo(TripleName, "z10", ""));
EXPECT_NE(STI, nullptr);
- std::unique_ptr<MCAsmInfo> MAI;
MAI.reset(TheTarget->createMCAsmInfo(*MRI, TripleName, MCOptions));
EXPECT_NE(MAI, nullptr);
-
- // Now we cast to our mocked up version of MCAsmInfo.
- MUPMAI.reset(static_cast<MockedUpMCAsmInfo *>(MAI.release()));
- // MUPMAI should "hold" MAI.
- EXPECT_NE(MUPMAI, nullptr);
- // After releasing, MAI should now be null.
- EXPECT_EQ(MAI, nullptr);
}
void setupCallToAsmParser(StringRef AsmStr) {
@@ -114,7 +78,7 @@ class SystemZAsmLexerTest : public ::testing::Test {
SrcMgr.AddNewSourceBuffer(std::move(Buffer), SMLoc());
EXPECT_EQ(Buffer, nullptr);
- Ctx.reset(new MCContext(Triple, MUPMAI.get(), MRI.get(), STI.get(), &SrcMgr,
+ Ctx.reset(new MCContext(Triple, MAI.get(), MRI.get(), STI.get(), &SrcMgr,
&MCOptions));
MOFI.reset(TheTarget->createMCObjectFileInfo(*Ctx, /*PIC=*/false,
/*LargeCodeModel=*/false));
@@ -122,7 +86,7 @@ class SystemZAsmLexerTest : public ::testing::Test {
Str.reset(TheTarget->createNullStreamer(*Ctx));
- Parser.reset(createMCAsmParser(SrcMgr, *Ctx, *Str, *MUPMAI));
+ Parser.reset(createMCAsmParser(SrcMgr, *Ctx, *Str, *MAI));
TargetAsmParser.reset(
TheTarget->createMCAsmParser(*STI, *Parser, *MII, MCOptions));
@@ -156,7 +120,17 @@ class SystemZAsmLexerTest : public ::testing::Test {
}
};
-TEST_F(SystemZAsmLexerTest, CheckDontRestrictCommentStringToStartOfStatement) {
+class SystemZAsmLexerLinux : public SystemZAsmLexerTest {
+protected:
+ SystemZAsmLexerLinux() : SystemZAsmLexerTest("s390x-ibm-linux") {}
+};
+
+class SystemZAsmLexerZOS : public SystemZAsmLexerTest {
+protected:
+ SystemZAsmLexerZOS() : SystemZAsmLexerTest("s390x-ibm-zos") {}
+};
+
+TEST_F(SystemZAsmLexerLinux, CheckDontRestrictCommentStringToStartOfStatement) {
StringRef AsmStr = "jne #-4";
// Setup.
@@ -170,12 +144,10 @@ TEST_F(SystemZAsmLexerTest, CheckDontRestrictCommentStringToStartOfStatement) {
lexAndCheckTokens(AsmStr /* "jne #-4" */, ExpectedTokens);
}
-// Testing MCAsmInfo's RestrictCommentStringToStartOfStatement attribute.
-TEST_F(SystemZAsmLexerTest, CheckRestrictCommentStringToStartOfStatement) {
+TEST_F(SystemZAsmLexerZOS, CheckRestrictCommentStringToStartOfStatement) {
StringRef AsmStr = "jne #-4";
// Setup.
- MUPMAI->setRestrictCommentStringToStartOfStatement(true);
setupCallToAsmParser(AsmStr);
// Lex initially to get the string.
@@ -187,17 +159,15 @@ TEST_F(SystemZAsmLexerTest, CheckRestrictCommentStringToStartOfStatement) {
// Minus - '-'
// Integer - '4'
SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::Identifier, AsmToken::Hash, AsmToken::Minus,
- AsmToken::Integer});
+ {AsmToken::Identifier, AsmToken::Space, AsmToken::Identifier});
lexAndCheckTokens(AsmStr /* "jne #-4" */, ExpectedTokens);
}
// Test HLASM Comment Syntax ('*')
-TEST_F(SystemZAsmLexerTest, CheckHLASMComment) {
+TEST_F(SystemZAsmLexerZOS, CheckHLASMComment) {
StringRef AsmStr = "* lhi 1,10";
// Setup.
- MUPMAI->setCommentString("*");
setupCallToAsmParser(AsmStr);
// Lex initially to get the string.
@@ -208,7 +178,7 @@ TEST_F(SystemZAsmLexerTest, CheckHLASMComment) {
lexAndCheckTokens(AsmStr /* "* lhi 1,10" */, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckHashDefault) {
+TEST_F(SystemZAsmLexerLinux, CheckHashDefault) {
StringRef AsmStr = "lh#123";
// Setup.
@@ -225,12 +195,11 @@ TEST_F(SystemZAsmLexerTest, CheckHashDefault) {
}
// Test if "#" is accepted as an Identifier
-TEST_F(SystemZAsmLexerTest, CheckAllowHashInIdentifier) {
+TEST_F(SystemZAsmLexerZOS, CheckAllowHashInIdentifier) {
StringRef AsmStr = "lh#123";
// Setup.
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setAllowHashInIdentifier(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
@@ -241,14 +210,11 @@ TEST_F(SystemZAsmLexerTest, CheckAllowHashInIdentifier) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAllowHashInIdentifier2) {
+TEST_F(SystemZAsmLexerZOS, CheckAllowHashInIdentifier2) {
StringRef AsmStr = "lh#12*3";
// Setup.
- MUPMAI->setCommentString("*");
- MUPMAI->setRestrictCommentStringToStartOfStatement(true);
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setAllowHashInIdentifier(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
@@ -262,7 +228,7 @@ TEST_F(SystemZAsmLexerTest, CheckAllowHashInIdentifier2) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, DontCheckStrictCommentString) {
+TEST_F(SystemZAsmLexerLinux, DontCheckStrictCommentString) {
StringRef AsmStr = "# abc\n/* def */// xyz";
// Setup.
@@ -277,113 +243,30 @@ TEST_F(SystemZAsmLexerTest, DontCheckStrictCommentString) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, DontCheckStrictCommentString2) {
- StringRef AsmStr = "# abc\n/* def */// xyz\n* rst";
-
- // Setup.
- MUPMAI->setCommentString("*");
- setupCallToAsmParser(AsmStr);
-
- // Lex initially to get the string.
- Parser->getLexer().Lex();
-
- SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::EndOfStatement, AsmToken::Comment, AsmToken::EndOfStatement,
- AsmToken::EndOfStatement, AsmToken::Eof});
- lexAndCheckTokens(AsmStr, ExpectedTokens);
-}
-
-TEST_F(SystemZAsmLexerTest, CheckStrictCommentString) {
+TEST_F(SystemZAsmLexerZOS, CheckStrictCommentString) {
StringRef AsmStr = "# abc\n/* def */// xyz";
// Setup.
- MUPMAI->setAllowAdditionalComments(false);
- setupCallToAsmParser(AsmStr);
-
- // Lex initially to get the string.
- Parser->getLexer().Lex();
-
- // "# abc" -> still treated as a comment, since CommentString
- // is set to "#"
- SmallVector<AsmToken::TokenKind> ExpectedTokens;
- ExpectedTokens.push_back(AsmToken::EndOfStatement); // "# abc\n"
- ExpectedTokens.push_back(AsmToken::Slash); // "/"
- ExpectedTokens.push_back(AsmToken::Star); // "*"
- ExpectedTokens.push_back(AsmToken::Identifier); // "def"
- ExpectedTokens.push_back(AsmToken::Star); // "*"
- ExpectedTokens.push_back(AsmToken::Slash); // "/"
- ExpectedTokens.push_back(AsmToken::Slash); // "/"
- ExpectedTokens.push_back(AsmToken::Slash); // "/"
- ExpectedTokens.push_back(AsmToken::Identifier); // "xyz"
- ExpectedTokens.push_back(AsmToken::EndOfStatement);
- ExpectedTokens.push_back(AsmToken::Eof);
-
- lexAndCheckTokens(AsmStr, ExpectedTokens);
-}
-
-TEST_F(SystemZAsmLexerTest, CheckStrictCommentString2) {
- StringRef AsmStr = "// abc";
-
- // Setup.
- MUPMAI->setAllowAdditionalComments(false);
- MUPMAI->setCommentString("//");
- setupCallToAsmParser(AsmStr);
-
- // Lex initially to get the string.
- Parser->getLexer().Lex();
-
- // "// abc" -> will still be treated as a comment because "//" is the
- // CommentString
- SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::EndOfStatement, AsmToken::Eof});
- lexAndCheckTokens(AsmStr /* "// abc" */, ExpectedTokens);
-}
-
-TEST_F(SystemZAsmLexerTest, CheckStrictCommentString3) {
- StringRef AsmStr = "/* abc */";
-
- // Setup.
- MUPMAI->setAllowAdditionalComments(false);
setupCallToAsmParser(AsmStr);
// Lex initially to get the string.
Parser->getLexer().Lex();
SmallVector<AsmToken::TokenKind> ExpectedTokens;
- ExpectedTokens.push_back(AsmToken::Slash);
- ExpectedTokens.push_back(AsmToken::Star);
- ExpectedTokens.push_back(AsmToken::Identifier);
- ExpectedTokens.push_back(AsmToken::Star);
- ExpectedTokens.push_back(AsmToken::Slash);
- ExpectedTokens.push_back(AsmToken::EndOfStatement);
- ExpectedTokens.push_back(AsmToken::Eof);
-
- lexAndCheckTokens(AsmStr, ExpectedTokens);
-}
-
-TEST_F(SystemZAsmLexerTest, CheckStrictCommentString4) {
- StringRef AsmStr = "# abc\n/* def */// xyz";
-
- // Setup.
- MUPMAI->setCommentString("*");
- MUPMAI->setAllowAdditionalComments(false);
- MUPMAI->setRestrictCommentStringToStartOfStatement(true);
- setupCallToAsmParser(AsmStr);
-
- // Lex initially to get the string.
- Parser->getLexer().Lex();
-
- SmallVector<AsmToken::TokenKind> ExpectedTokens;
- ExpectedTokens.push_back(AsmToken::Hash); // "#"
+ ExpectedTokens.push_back(AsmToken::Identifier); // "#"
+ ExpectedTokens.push_back(AsmToken::Space); // " "
ExpectedTokens.push_back(AsmToken::Identifier); // "abc"
ExpectedTokens.push_back(AsmToken::EndOfStatement); // "\n"
ExpectedTokens.push_back(AsmToken::Slash); // "/"
ExpectedTokens.push_back(AsmToken::Star); // "*"
+ ExpectedTokens.push_back(AsmToken::Space); // " "
ExpectedTokens.push_back(AsmToken::Identifier); // "def"
+ ExpectedTokens.push_back(AsmToken::Space); // " "
ExpectedTokens.push_back(AsmToken::Star); // "*"
ExpectedTokens.push_back(AsmToken::Slash); // "/"
ExpectedTokens.push_back(AsmToken::Slash); // "/"
ExpectedTokens.push_back(AsmToken::Slash); // "/"
+ ExpectedTokens.push_back(AsmToken::Space); // " "
ExpectedTokens.push_back(AsmToken::Identifier); // "xyz"
ExpectedTokens.push_back(AsmToken::EndOfStatement);
ExpectedTokens.push_back(AsmToken::Eof);
@@ -391,34 +274,11 @@ TEST_F(SystemZAsmLexerTest, CheckStrictCommentString4) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckStrictCommentString5) {
- StringRef AsmStr = "#abc\n/* def */// xyz";
-
- // Setup.
- MUPMAI->setCommentString("*");
- MUPMAI->setAllowAdditionalComments(false);
- setupCallToAsmParser(AsmStr);
-
- // Lex initially to get the string.
- Parser->getLexer().Lex();
-
- SmallVector<AsmToken::TokenKind> ExpectedTokens;
- ExpectedTokens.push_back(AsmToken::Hash); // "#"
- ExpectedTokens.push_back(AsmToken::Identifier); // "abc"
- ExpectedTokens.push_back(AsmToken::EndOfStatement); // "\n"
- ExpectedTokens.push_back(AsmToken::Slash); // "/"
- ExpectedTokens.push_back(AsmToken::EndOfStatement); // "* def */// xyz"
- ExpectedTokens.push_back(AsmToken::Eof);
-
- lexAndCheckTokens(AsmStr, ExpectedTokens);
-}
-
-TEST_F(SystemZAsmLexerTest, CheckValidHLASMIntegers) {
+TEST_F(SystemZAsmLexerZOS, CheckValidHLASMIntegers) {
StringRef AsmStr = "123\n000123\n1999\n007\n12300\n12021\n";
// StringRef AsmStr = "123";
// Setup.
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setLexHLASMIntegers(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
@@ -428,12 +288,11 @@ TEST_F(SystemZAsmLexerTest, CheckValidHLASMIntegers) {
lexAndCheckIntegerTokensAndValues(AsmStr, ExpectedValues);
}
-TEST_F(SystemZAsmLexerTest, CheckInvalidHLASMIntegers) {
+TEST_F(SystemZAsmLexerZOS, CheckInvalidHLASMIntegers) {
StringRef AsmStr = "0b0101\n0xDEADBEEF\nfffh\n.133\n";
// Setup.
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setLexHLASMIntegers(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
@@ -453,7 +312,7 @@ TEST_F(SystemZAsmLexerTest, CheckInvalidHLASMIntegers) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckDefaultIntegers) {
+TEST_F(SystemZAsmLexerLinux, CheckDefaultIntegers) {
StringRef AsmStr = "0b0101\n0xDEADBEEF\nfffh\n";
// Setup.
@@ -466,7 +325,7 @@ TEST_F(SystemZAsmLexerTest, CheckDefaultIntegers) {
lexAndCheckIntegerTokensAndValues(AsmStr, ExpectedValues);
}
-TEST_F(SystemZAsmLexerTest, CheckDefaultFloats) {
+TEST_F(SystemZAsmLexerLinux, CheckDefaultFloats) {
StringRef AsmStr = "0.333\n1.3\n2.5\n3.0\n";
// Setup.
@@ -485,7 +344,7 @@ TEST_F(SystemZAsmLexerTest, CheckDefaultFloats) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckDefaultQuestionAtStartOfIdentifier) {
+TEST_F(SystemZAsmLexerLinux, CheckDefaultQuestionAtStartOfIdentifier) {
StringRef AsmStr = "?lh1?23";
// Setup.
@@ -500,26 +359,10 @@ TEST_F(SystemZAsmLexerTest, CheckDefaultQuestionAtStartOfIdentifier) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAcceptQuestionAtStartOfIdentifier) {
- StringRef AsmStr = "?????lh1?23";
-
- // Setup.
- MUPMAI->setAllowQuestionAtStartOfIdentifier(true);
- setupCallToAsmParser(AsmStr);
-
- // Lex initially to get the string.
- Parser->getLexer().Lex();
-
- SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::Identifier, AsmToken::EndOfStatement, AsmToken::Eof});
- lexAndCheckTokens(AsmStr, ExpectedTokens);
-}
-
-TEST_F(SystemZAsmLexerTest, CheckDefaultAtAtStartOfIdentifier) {
+TEST_F(SystemZAsmLexerLinux, CheckDefaultAtAtStartOfIdentifier) {
StringRef AsmStr = "@@lh1?23";
// Setup.
- MUPMAI->setAllowQuestionAtStartOfIdentifier(true);
setupCallToAsmParser(AsmStr);
// Lex initially to get the string.
@@ -531,11 +374,10 @@ TEST_F(SystemZAsmLexerTest, CheckDefaultAtAtStartOfIdentifier) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAcceptAtAtStartOfIdentifier) {
+TEST_F(SystemZAsmLexerZOS, CheckAcceptAtAtStartOfIdentifier) {
StringRef AsmStr = "@@lh1?23";
// Setup.
- MUPMAI->setAllowAtAtStartOfIdentifier(true);
setupCallToAsmParser(AsmStr);
// Lex initially to get the string.
@@ -546,24 +388,7 @@ TEST_F(SystemZAsmLexerTest, CheckAcceptAtAtStartOfIdentifier) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAccpetAtAtStartOfIdentifier2) {
- StringRef AsmStr = "@@lj1?23";
-
- // Setup.
- MUPMAI->setCommentString("@");
- MUPMAI->setAllowAtAtStartOfIdentifier(true);
- setupCallToAsmParser(AsmStr);
-
- // Lex initially to get the string.
- Parser->getLexer().Lex();
-
- // "@@lj1?23" -> still lexed as a comment as that takes precedence.
- SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::EndOfStatement, AsmToken::Eof});
- lexAndCheckTokens(AsmStr, ExpectedTokens);
-}
-
-TEST_F(SystemZAsmLexerTest, CheckDefaultDollarAtStartOfIdentifier) {
+TEST_F(SystemZAsmLexerLinux, CheckDefaultDollarAtStartOfIdentifier) {
StringRef AsmStr = "$$ac$c";
// Setup.
@@ -578,11 +403,10 @@ TEST_F(SystemZAsmLexerTest, CheckDefaultDollarAtStartOfIdentifier) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAcceptDollarAtStartOfIdentifier) {
+TEST_F(SystemZAsmLexerZOS, CheckAcceptDollarAtStartOfIdentifier) {
StringRef AsmStr = "$$ab$c";
// Setup.
- MUPMAI->setAllowDollarAtStartOfIdentifier(true);
setupCallToAsmParser(AsmStr);
// Lex initially to get the string.
@@ -593,15 +417,11 @@ TEST_F(SystemZAsmLexerTest, CheckAcceptDollarAtStartOfIdentifier) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAcceptHashAtStartOfIdentifier) {
+TEST_F(SystemZAsmLexerZOS, CheckAcceptHashAtStartOfIdentifier) {
StringRef AsmStr = "##a#b$c";
// Setup.
- MUPMAI->setAllowHashAtStartOfIdentifier(true);
- MUPMAI->setCommentString("*");
- MUPMAI->setAllowAdditionalComments(false);
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setAllowHashInIdentifier(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
@@ -611,13 +431,11 @@ TEST_F(SystemZAsmLexerTest, CheckAcceptHashAtStartOfIdentifier) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAcceptHashAtStartOfIdentifier2) {
+TEST_F(SystemZAsmLexerLinux, CheckAcceptHashAtStartOfIdentifier2) {
StringRef AsmStr = "##a#b$c";
// Setup.
- MUPMAI->setAllowHashAtStartOfIdentifier(true);
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setAllowHashInIdentifier(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
@@ -630,36 +448,25 @@ TEST_F(SystemZAsmLexerTest, CheckAcceptHashAtStartOfIdentifier2) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAcceptHashAtStartOfIdentifier3) {
+TEST_F(SystemZAsmLexerZOS, CheckAcceptHashAtStartOfIdentifier3) {
StringRef AsmStr = "##a#b$c";
// Setup.
- MUPMAI->setAllowHashAtStartOfIdentifier(true);
- MUPMAI->setCommentString("*");
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setAllowHashInIdentifier(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
- // By default, the AsmLexer treats strings that start with "#"
- // as a line comment.
- // Hence, "##a$b$c" is lexed as a line comment irrespective
- // of whether the AllowHashAtStartOfIdentifier attribute is set to true.
SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::EndOfStatement, AsmToken::Eof});
+ {AsmToken::Identifier, AsmToken::EndOfStatement, AsmToken::Eof});
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckAcceptHashAtStartOfIdentifier4) {
+TEST_F(SystemZAsmLexerZOS, CheckAcceptHashAtStartOfIdentifier4) {
StringRef AsmStr = "##a#b$c";
// Setup.
- MUPMAI->setAllowHashAtStartOfIdentifier(true);
- MUPMAI->setCommentString("*");
- MUPMAI->setAllowAdditionalComments(false);
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setAllowHashInIdentifier(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
@@ -674,11 +481,10 @@ TEST_F(SystemZAsmLexerTest, CheckAcceptHashAtStartOfIdentifier4) {
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckRejectDotAsCurrentPC) {
+TEST_F(SystemZAsmLexerZOS, CheckRejectDotAsCurrentPC) {
StringRef AsmStr = ".-4";
// Setup.
- MUPMAI->setAllowDotIsPC(false);
setupCallToAsmParser(AsmStr);
// Lex initially to get the string.
@@ -690,7 +496,7 @@ TEST_F(SystemZAsmLexerTest, CheckRejectDotAsCurrentPC) {
EXPECT_EQ(Parser->hasPendingError(), true);
}
-TEST_F(SystemZAsmLexerTest, CheckRejectStarAsCurrentPC) {
+TEST_F(SystemZAsmLexerLinux, CheckRejectStarAsCurrentPC) {
StringRef AsmStr = "*-4";
// Setup.
@@ -705,47 +511,53 @@ TEST_F(SystemZAsmLexerTest, CheckRejectStarAsCurrentPC) {
EXPECT_EQ(Parser->hasPendingError(), true);
}
-TEST_F(SystemZAsmLexerTest, CheckRejectCharLiterals) {
+TEST_F(SystemZAsmLexerZOS, CheckRejectCharLiterals) {
StringRef AsmStr = "abc 'd'";
// Setup.
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setLexHLASMStrings(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::Identifier, AsmToken::Error, AsmToken::Error,
+ {AsmToken::Identifier, AsmToken::Space, AsmToken::Error, AsmToken::Error,
AsmToken::EndOfStatement, AsmToken::Eof});
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckRejectStringLiterals) {
+TEST_F(SystemZAsmLexerZOS, CheckRejectStringLiterals) {
StringRef AsmStr = "abc \"ef\"";
// Setup.
setupCallToAsmParser(AsmStr);
- Parser->getLexer().setLexHLASMStrings(true);
// Lex initially to get the string.
Parser->getLexer().Lex();
SmallVector<AsmToken::TokenKind> ExpectedTokens(
- {AsmToken::Identifier, AsmToken::Error, AsmToken::Identifier,
- AsmToken::Error, AsmToken::EndOfStatement, AsmToken::Eof});
+ {AsmToken::Identifier, AsmToken::Space, AsmToken::Error,
+ AsmToken::Identifier, AsmToken::Error, AsmToken::EndOfStatement,
+ AsmToken::Eof});
lexAndCheckTokens(AsmStr, ExpectedTokens);
}
-TEST_F(SystemZAsmLexerTest, CheckPrintAcceptableSymbol) {
+TEST_F(SystemZAsmLexerZOS, CheckPrintAcceptableSymbol) {
+ std::string AsmStr = "ab13_$.@";
+ EXPECT_EQ(true, MAI->isValidUnquotedName(AsmStr));
+ AsmStr += "#";
+ EXPECT_EQ(true, MAI->isValidUnquotedName(AsmStr));
+}
+
+TEST_F(SystemZAsmLexerLinux, CheckPrintAcceptableSymbol) {
std::string AsmStr = "ab13_$.@";
- EXPECT_EQ(true, MUPMAI->isValidUnquotedName(AsmStr));
+ EXPECT_EQ(true, MAI->isValidUnquotedName(AsmStr));
AsmStr += "#";
- EXPECT_EQ(false, MUPMAI->isValidUnquotedName(AsmStr));
+ EXPECT_EQ(false, MAI->isValidUnquotedName(AsmStr));
}
-TEST_F(SystemZAsmLexerTest, CheckLabelCaseUpperCase2) {
- StringRef AsmStr = "label\nlabel";
+TEST_F(SystemZAsmLexerZOS, CheckLabelCaseUpperCase) {
+ StringRef AsmStr = "label";
// Setup.
setupCallToAsmParser(AsmStr);
@@ -760,19 +572,25 @@ TEST_F(SystemZAsmLexerTest, CheckLabelCaseUpperCase2) {
const MCSymbolRefExpr *SymbolExpr = dyn_cast<MCSymbolRefExpr>(Expr);
EXPECT_NE(SymbolExpr, nullptr);
EXPECT_NE(&SymbolExpr->getSymbol(), nullptr);
- EXPECT_EQ((&SymbolExpr->getSymbol())->getName(), StringRef("label"));
+ EXPECT_EQ((&SymbolExpr->getSymbol())->getName(), StringRef("LABEL"));
+}
- // Lex the end of statement token.
- Parser->getLexer().Lex();
+TEST_F(SystemZAsmLexerLinux, CheckLabelUpperCase2) {
+ StringRef AsmStr = "label";
- MUPMAI->setEmitLabelsInUpperCase(true);
+ // Setup.
+ setupCallToAsmParser(AsmStr);
- ParsePrimaryExpr = Parser->parseExpression(Expr);
+ // Lex initially to get the string.
+ Parser->getLexer().Lex();
+
+ const MCExpr *Expr;
+ bool ParsePrimaryExpr = Parser->parseExpression(Expr);
EXPECT_EQ(ParsePrimaryExpr, false);
- SymbolExpr = dyn_cast<MCSymbolRefExpr>(Expr);
+ const MCSymbolRefExpr *SymbolExpr = dyn_cast<MCSymbolRefExpr>(Expr);
EXPECT_NE(SymbolExpr, nullptr);
EXPECT_NE(&SymbolExpr->getSymbol(), nullptr);
- EXPECT_EQ((&SymbolExpr->getSymbol())->getName(), StringRef("LABEL"));
+ EXPECT_EQ((&SymbolExpr->getSymbol())->getName(), StringRef("label"));
}
} // end anonymous namespace
More information about the llvm-commits
mailing list