[LLVMdev] 2.5 Pre-release1 available for testing

[Duncan Sands]

Hi Jeff,

> > 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.

which compiler version are you using?  If this patch is applied,
does everything work fine if you build with that compiler or are
there other issues?

Thanks,

Duncan.

>
> 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:
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
>