[cfe-commits] r150294 - in /cfe/trunk/lib/Driver: ToolChains.cpp ToolChains.h
Chandler Carruth
chandlerc at gmail.com
Fri Feb 10 19:31:12 PST 2012
Author: chandlerc
Date: Fri Feb 10 21:31:12 2012
New Revision: 150294
URL: http://llvm.org/viewvc/llvm-project?rev=150294&view=rev
Log:
Begin refactoring to use the newly added triple predicates for
simplicity. Also addresses a FIXME, although not one that could be
observed.
Modified:
cfe/trunk/lib/Driver/ToolChains.cpp
cfe/trunk/lib/Driver/ToolChains.h
Modified: cfe/trunk/lib/Driver/ToolChains.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.cpp?rev=150294&r1=150293&r2=150294&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains.cpp Fri Feb 10 21:31:12 2012
@@ -2001,12 +2001,7 @@
// to the link paths.
path_list &Paths = getFilePaths();
- const bool Is32Bits = (Arch == llvm::Triple::x86 ||
- Arch == llvm::Triple::mips ||
- Arch == llvm::Triple::mipsel ||
- Arch == llvm::Triple::ppc);
-
- const std::string Multilib = Is32Bits ? "lib32" : "lib64";
+ const std::string Multilib = Triple.isArch32Bit() ? "lib32" : "lib64";
const std::string MultiarchTriple = getMultiarchTriple(Triple, SysRoot);
// Add the multilib suffixed paths where they are available.
Modified: cfe/trunk/lib/Driver/ToolChains.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/ToolChains.h?rev=150294&r1=150293&r2=150294&view=diff
==============================================================================
--- cfe/trunk/lib/Driver/ToolChains.h (original)
+++ cfe/trunk/lib/Driver/ToolChains.h Fri Feb 10 21:31:12 2012
@@ -134,13 +134,10 @@
/// @{
/// \brief Check whether the target triple's architecture is 64-bits.
- bool isTarget64Bit() const {
- return (getTriple().getArch() == llvm::Triple::x86_64 ||
- getTriple().getArch() == llvm::Triple::ppc64);
- }
+ bool isTarget64Bit() const { return getTriple().isArch64Bit(); }
+
/// \brief Check whether the target triple's architecture is 32-bits.
- /// FIXME: This should likely do more than just negate the 64-bit query.
- bool isTarget32Bit() const { return !isTarget64Bit(); }
+ bool isTarget32Bit() const { return getTriple().isArch32Bit(); }
/// @}
};
More information about the cfe-commits
mailing list