[LLVMdev] 2.5 Pre-release1 available for testing

[Jeff Fifield]

Michel Salim wrote:
> On Fri, Feb 6, 2009 at 8:42 PM, Tanya Lattner <tonic at nondot.org> wrote:
>> LLVMers,
>>
>> The 2.5 pre-release is available for testing:
>> http://llvm.org/prereleases/2.5/
>>
> I'm updating the Fedora packaging of LLVM, and with the 02/20
> prerelease it fails to build on ppc64:
> 
> http://koji.fedoraproject.org/koji/taskinfo?taskID=1148023
> 
> make[1]: Entering directory `/builddir/build/BUILD/llvm-2.5/lib/VMCore'
> llvm[1]: Building Intrinsics.gen.tmp from Intrinsics.td
> tblgen: IntrinsicEmitter.cpp:137: void
> EmitTypeForValueType(std::ostream&, llvm::MVT::SimpleValueType):
> Assertion `false && "Unsupported ValueType!"' failed.
> make[1]: Leaving directory `/builddir/build/BUILD/llvm-2.5/lib/VMCore'
> make[1]: *** [/builddir/build/BUILD/llvm-2.5/lib/VMCore/Release/Intrinsics.gen.tmp]
> Aborted
> make: *** [all] Error 1
> 
> The compiler flags used are in the build log as well.
> 
> Thanks,
> 
> 

I ran into the same problem and fixed it by forcing the
MVT::SimpleValueType enum to be 64 bits so that all of the types
in the union later in the class are the same size.  I tested this
on ppc64 and x86_64.

Index: include/llvm/CodeGen/ValueTypes.h
===================================================================
--- include/llvm/CodeGen/ValueTypes.h	(revision 66504)
+++ include/llvm/CodeGen/ValueTypes.h	(working copy)
@@ -21,6 +21,8 @@
  #include "llvm/Support/DataTypes.h"
  #include "llvm/Support/MathExtras.h"

+static const uintptr_t minus_one = -1;
+
  namespace llvm {
    class Type;

@@ -92,7 +94,10 @@
        iPTR           =  255,

        // LastSimpleValueType - The greatest valid SimpleValueTypevalue.
-      LastSimpleValueType = 255
+      LastSimpleValueType = 255,
+
+      // force the size of the enum to be sizeof(uintptr_t)
+      _BigValue = minus_one
      };

    private: