Quick and dirty (!) fix to make sure we use powerpc in triples.
- PR3922
- I have a clean solution for this in flight, but it may take a while
to come to fruition so we'll take a quick fix for now.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@68241 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Driver/Driver.cpp b/lib/Driver/Driver.cpp
index 84e4cea..b59768d 100644
--- a/lib/Driver/Driver.cpp
+++ b/lib/Driver/Driver.cpp
@@ -1065,8 +1065,10 @@
Arch = "i386";
else if (Arch == "amd64")
Arch = "x86_64";
- else if (Arch == "powerpc" || Arch == "Power Macintosh")
- Arch = "ppc";
+ else if (Arch == "ppc" || Arch == "Power Macintosh")
+ Arch = "powerpc";
+ else if (Arch == "ppc64")
+ Arch = "powerpc64";
if (memcmp(&OS[0], "darwin", 6) == 0)
return createDarwinHostInfo(*this, Arch.c_str(), Platform.c_str(),
@@ -1080,7 +1082,14 @@
}
bool Driver::ShouldUseClangCompiler(const Compilation &C, const JobAction &JA,
- const std::string &ArchName) const {
+ const std::string &ArchNameStr) const {
+ // FIXME: Remove this hack.
+ const char *ArchName = ArchNameStr.c_str();
+ if (ArchNameStr == "powerpc")
+ ArchName = "ppc";
+ else if (ArchNameStr == "powerpc64")
+ ArchName = "ppc64";
+
// Check if user requested no clang, or clang doesn't understand
// this type (we only handle single inputs for now).
if (!CCCUseClang || JA.size() != 1 ||
diff --git a/lib/Driver/HostInfo.cpp b/lib/Driver/HostInfo.cpp
index 2ce06f5..8db18d4 100644
--- a/lib/Driver/HostInfo.cpp
+++ b/lib/Driver/HostInfo.cpp
@@ -77,7 +77,7 @@
: HostInfo(D, _Arch, _Platform, _OS) {
assert((getArchName() == "i386" || getArchName() == "x86_64" ||
- getArchName() == "ppc" || getArchName() == "ppc64") &&
+ getArchName() == "powerpc" || getArchName() == "powerpc64") &&
"Unknown Darwin arch.");
assert(memcmp(&getOSName()[0], "darwin", 6) == 0 &&
@@ -117,11 +117,17 @@
if (getArchName() == "i386" || getArchName() == "x86_64") {
ArchName =
(A->getOption().getId() == options::OPT_m32) ? "i386" : "x86_64";
- } else if (getArchName() == "ppc" || getArchName() == "ppc64") {
- ArchName =
- (A->getOption().getId() == options::OPT_m32) ? "ppc" : "ppc64";
+ } else if (getArchName() == "powerpc" || getArchName() == "powerpc64") {
+ ArchName = (A->getOption().getId() == options::OPT_m32) ? "powerpc" :
+ "powerpc64";
}
}
+ } else {
+ // Normalize arch name; we shouldn't be doing this here.
+ if (strcmp(ArchName, "ppc") == 0)
+ ArchName = "powerpc";
+ else if (strcmp(ArchName, "ppc64") == 0)
+ ArchName = "powerpc64";
}
ToolChain *&TC = ToolChains[ArchName];
@@ -190,9 +196,9 @@
if (getArchName() == "i386" || getArchName() == "x86_64") {
ArchName =
(A->getOption().getId() == options::OPT_m32) ? "i386" : "x86_64";
- } else if (getArchName() == "ppc" || getArchName() == "ppc64") {
+ } else if (getArchName() == "powerpc" || getArchName() == "powerpc64") {
ArchName =
- (A->getOption().getId() == options::OPT_m32) ? "ppc" : "ppc64";
+ (A->getOption().getId() == options::OPT_m32) ? "powerpc" : "powerpc64";
}
}
diff --git a/lib/Driver/Tools.cpp b/lib/Driver/Tools.cpp
index e4fff9c..b8c07cc 100644
--- a/lib/Driver/Tools.cpp
+++ b/lib/Driver/Tools.cpp
@@ -453,7 +453,14 @@
// If using a driver driver, force the arch.
if (getToolChain().getHost().useDriverDriver()) {
CmdArgs.push_back("-arch");
- CmdArgs.push_back(getToolChain().getArchName().c_str());
+
+ // FIXME: Remove these special cases.
+ const char *Str = getToolChain().getArchName().c_str();
+ if (strcmp(Str, "powerpc") == 0)
+ Str = "ppc";
+ else if (strcmp(Str, "powerpc64") == 0)
+ Str = "ppc64";
+ CmdArgs.push_back(Str);
}
if (Output.isPipe()) {