[PATCH] [clang][MIPS] Removing __SIZEOF_INT128__ macro for MIPS64

David Majnemer david.majnemer at gmail.com
Thu Dec 4 01:36:29 PST 2014


I was able to #include <limits.h> with just the following diff:

diff --git a/lib/Basic/Targets.cpp b/lib/Basic/Targets.cpp
index b28808b..67bff45 100644
--- a/lib/Basic/Targets.cpp
+++ b/lib/Basic/Targets.cpp
@@ -5971,7 +5971,7 @@ public:
     NumAliases = llvm::array_lengthof(GCCRegAliases);
   }

-  bool hasInt128Type() const override { return true; }
+  bool hasInt128Type() const override { return false; }
 };

 class Mips64EBTargetInfo : public Mips64TargetInfoBase {


I tested it with: echo '#include <limits.h>' |
~/llvm/Debug+Asserts/bin/clang -x c -target mips64-none-none - -fsyntax-only


On Thu, Dec 4, 2014 at 1:02 AM, Sagar Thakur <Sagar.Thakur at imgtec.com>
wrote:

> Hi @atanasyan,
>
> I don't have commit access. Can you commit it for me ?
>
> http://reviews.llvm.org/D6402
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20141204/71839369/attachment.html>


More information about the cfe-commits mailing list