Setting SwitchToSectionDirective properly in the MASM backend permits a bunch

of code to be unified.

llvm-svn: 28191
This commit is contained in:
Chris Lattner 2006-05-09 05:23:12 +00:00
parent 0b7acaf027
commit 6341df8069
1 changed files with 15 additions and 22 deletions

View File

@ -102,34 +102,27 @@ void AsmPrinter::SwitchToTextSection(const char *NewSection,
void AsmPrinter::SwitchToDataSection(const char *NewSection,
const GlobalValue *GV) {
std::string NS;
if (GV && GV->hasSection())
NS = SwitchToSectionDirective + GV->getSection();
else
NS = NewSection;
// If we're already in this section, we're done.
if (CurrentSection == NS) return;
// Microsoft ML/MASM has a fundamentally different approach to handling
// sections.
if (MLSections) {
if (GV && GV->hasSection())
NS = GV->getSection();
else
NS = NewSection;
if (CurrentSection != NS) {
if (!CurrentSection.empty())
O << CurrentSection << "\tends\n\n";
CurrentSection = NS;
if (!CurrentSection.empty())
O << CurrentSection << "\tsegment 'DATA'\n";
}
if (!CurrentSection.empty())
O << CurrentSection << "\tends\n\n";
CurrentSection = NS;
if (!CurrentSection.empty())
O << CurrentSection << "\tsegment 'DATA'\n";
} else {
if (GV && GV->hasSection())
NS = SwitchToSectionDirective + GV->getSection();
else
NS = NewSection;
if (CurrentSection != NS) {
CurrentSection = NS;
if (!CurrentSection.empty())
O << CurrentSection << '\n';
}
CurrentSection = NS;
if (!CurrentSection.empty())
O << CurrentSection << '\n';
}
}