mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-12-23 03:59:12 +00:00
Revise r126127 to address Daniel's comments. <rdar://problem/9012638>
llvm-svn: 127939
This commit is contained in:
parent
3520263009
commit
4a2b1ca9c1
@ -157,10 +157,12 @@ public:
|
||||
/// flavored .s file. If successful, this fills in the specified Out
|
||||
/// parameters and returns an empty string. When an invalid section
|
||||
/// specifier is present, this returns a string indicating the problem.
|
||||
/// If no TAA was parsed, TAA is not altered, and TAAWasSet becomes false.
|
||||
static std::string ParseSectionSpecifier(StringRef Spec, // In.
|
||||
StringRef &Segment, // Out.
|
||||
StringRef &Section, // Out.
|
||||
unsigned &TAA, // Out.
|
||||
bool &TAAParsed, // Out.
|
||||
unsigned &StubSize); // Out.
|
||||
|
||||
virtual void PrintSwitchToSection(const MCAsmInfo &MAI,
|
||||
|
@ -641,10 +641,11 @@ getExplicitSectionGlobal(const GlobalValue *GV, SectionKind Kind,
|
||||
Mangler *Mang, const TargetMachine &TM) const {
|
||||
// Parse the section specifier and create it if valid.
|
||||
StringRef Segment, Section;
|
||||
unsigned TAA = (unsigned)MCSectionMachO::SECTION_ATTRIBUTES, StubSize = 0;
|
||||
unsigned TAA = 0, StubSize = 0;
|
||||
bool TAAParsed;
|
||||
std::string ErrorCode =
|
||||
MCSectionMachO::ParseSectionSpecifier(GV->getSection(), Segment, Section,
|
||||
TAA, StubSize);
|
||||
TAA, TAAParsed, StubSize);
|
||||
if (!ErrorCode.empty()) {
|
||||
// If invalid, report the error with report_fatal_error.
|
||||
report_fatal_error("Global variable '" + GV->getNameStr() +
|
||||
@ -654,17 +655,13 @@ getExplicitSectionGlobal(const GlobalValue *GV, SectionKind Kind,
|
||||
return DataSection;
|
||||
}
|
||||
|
||||
bool TAAWasSet = (TAA != MCSectionMachO::SECTION_ATTRIBUTES);
|
||||
if (!TAAWasSet)
|
||||
TAA = 0; // Sensible default if this is a new section.
|
||||
|
||||
// Get the section.
|
||||
const MCSectionMachO *S =
|
||||
getContext().getMachOSection(Segment, Section, TAA, StubSize, Kind);
|
||||
|
||||
// If TAA wasn't set by ParseSectionSpecifier() above,
|
||||
// use the value returned by getMachOSection() as a default.
|
||||
if (!TAAWasSet)
|
||||
if (!TAAParsed)
|
||||
TAA = S->getTypeAndAttributes();
|
||||
|
||||
// Okay, now that we got the section, verify that the TAA & StubSize agree.
|
||||
|
@ -435,10 +435,11 @@ bool DarwinAsmParser::ParseDirectiveSection(StringRef, SMLoc) {
|
||||
|
||||
StringRef Segment, Section;
|
||||
unsigned StubSize;
|
||||
unsigned TAA = 0;
|
||||
unsigned TAA;
|
||||
bool TAAParsed;
|
||||
std::string ErrorStr =
|
||||
MCSectionMachO::ParseSectionSpecifier(SectionSpec, Segment, Section,
|
||||
TAA, StubSize);
|
||||
TAA, TAAParsed, StubSize);
|
||||
|
||||
if (!ErrorStr.empty())
|
||||
return Error(Loc, ErrorStr.c_str());
|
||||
|
@ -180,7 +180,9 @@ std::string MCSectionMachO::ParseSectionSpecifier(StringRef Spec, // In.
|
||||
StringRef &Segment, // Out.
|
||||
StringRef &Section, // Out.
|
||||
unsigned &TAA, // Out.
|
||||
bool &TAAParsed, // Out.
|
||||
unsigned &StubSize) { // Out.
|
||||
TAAParsed = false;
|
||||
// Find the first comma.
|
||||
std::pair<StringRef, StringRef> Comma = Spec.split(',');
|
||||
|
||||
@ -211,6 +213,7 @@ std::string MCSectionMachO::ParseSectionSpecifier(StringRef Spec, // In.
|
||||
"between 1 and 16 characters";
|
||||
|
||||
// If there is no comma after the section, we're done.
|
||||
TAA = 0;
|
||||
StubSize = 0;
|
||||
if (Comma.second.empty())
|
||||
return "";
|
||||
@ -235,6 +238,7 @@ std::string MCSectionMachO::ParseSectionSpecifier(StringRef Spec, // In.
|
||||
|
||||
// Remember the TypeID.
|
||||
TAA = TypeID;
|
||||
TAAParsed = true;
|
||||
|
||||
// If we have no comma after the section type, there are no attributes.
|
||||
if (Comma.second.empty()) {
|
||||
|
Loading…
Reference in New Issue
Block a user