[cfe-commits] r134392 - /cfe/trunk/lib/Basic/Targets.cpp

[Joerg Sonnenberger]

On Mon, Jul 04, 2011 at 06:03:37PM -0700, Eli Friedman wrote:
> On Mon, Jul 4, 2011 at 2:57 PM, Joerg Sonnenberger <joerg at bec.de> wrote:
> > Author: joerg
> > Date: Mon Jul  4 16:57:55 2011
> > New Revision: 134392
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=134392&view=rev
> > Log:
> > On PowerPC, both FreeBSD and NetBSD use ints for (s)size_t
> >
> > Modified:
> >    cfe/trunk/lib/Basic/Targets.cpp
> >
> > Modified: cfe/trunk/lib/Basic/Targets.cpp
> > URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Basic/Targets.cpp?rev=134392&r1=134391&r2=134392&view=diff
> > ==============================================================================
> > --- cfe/trunk/lib/Basic/Targets.cpp (original)
> > +++ cfe/trunk/lib/Basic/Targets.cpp Mon Jul  4 16:57:55 2011
> > @@ -802,8 +802,12 @@
> >     DescriptionString = "E-p:32:32:32-i1:8:8-i8:8:8-i16:16:16-i32:32:32-"
> >                         "i64:64:64-f32:32:32-f64:64:64-v128:128:128-n32";
> >
> > -    if (getTriple().getOS() == llvm::Triple::FreeBSD)
> > +    switch (getTriple().getOS()) {
> > +    case llvm::Triple::FreeBSD:
> > +    case llvm::Triple::NetBSD:
> >         SizeType = UnsignedInt;
> > +        break;
> > +    }
> >   }
> >
> >   virtual const char *getVAListDeclaration() const {
> 
> You might want to check whether PtrDiffType and IntPtrType are being
> defined correctly as well...

I am aware of them, but the above actually created issues, so it
was the first item on the list.

Joerg