[llvm] r267589 - Fix warnings and -Werror build on clang.

[Zachary Turner via llvm-commits]

Strange, I didn't see them on my side.  I will fix those two up as well.

On Tue, Apr 26, 2016 at 1:27 PM Justin Bogner <mail at justinbogner.com> wrote:

> Zachary Turner via llvm-commits <llvm-commits at lists.llvm.org> writes:
> > Author: zturner
> > Date: Tue Apr 26 14:24:10 2016
> > New Revision: 267589
> >
> > URL: http://llvm.org/viewvc/llvm-project?rev=267589&view=rev
> > Log:
> > Fix warnings and -Werror build on clang.
>
> Seems like this isn't quite sufficient - I'm still getting warnings for
> unused variables BuildNewFormatMask and BuildNewFormatShift locally.
>
> > Modified:
> >     llvm/trunk/include/llvm/DebugInfo/PDB/Raw/PDBDbiStream.h
> >     llvm/trunk/lib/DebugInfo/PDB/Raw/PDBDbiStream.cpp
> >
> > Modified: llvm/trunk/include/llvm/DebugInfo/PDB/Raw/PDBDbiStream.h
> > URL:
> >
> http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/DebugInfo/PDB/Raw/PDBDbiStream.h?rev=267589&r1=267588&r2=267589&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/include/llvm/DebugInfo/PDB/Raw/PDBDbiStream.h (original)
> > +++ llvm/trunk/include/llvm/DebugInfo/PDB/Raw/PDBDbiStream.h Tue Apr 26
> 14:24:10 2016
> > @@ -43,8 +43,8 @@ public:
> >    PDB_Machine getMachineType() const;
> >
> >  private:
> > -  PDBStream Stream;
> >    PDBFile &Pdb;
> > +  PDBStream Stream;
> >    std::unique_ptr<HeaderInfo> Header;
> >  };
> >  }
> >
> > Modified: llvm/trunk/lib/DebugInfo/PDB/Raw/PDBDbiStream.cpp
> > URL:
> http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/DebugInfo/PDB/Raw/PDBDbiStream.cpp?rev=267589&r1=267588&r2=267589&view=diff
> >
> ==============================================================================
> > --- llvm/trunk/lib/DebugInfo/PDB/Raw/PDBDbiStream.cpp (original)
> > +++ llvm/trunk/lib/DebugInfo/PDB/Raw/PDBDbiStream.cpp Tue Apr 26
> 14:24:10 2016
> > @@ -47,7 +47,7 @@ const uint16_t BuildNewFormatShift = 15;
> >  }
> >
> >  struct PDBDbiStream::HeaderInfo {
> > -  ulittle32_t VersionSignature;
> > +  little32_t VersionSignature;
> >    ulittle32_t VersionHeader;
> >    ulittle32_t Age; // Should match PDBInfoStream.
> >    ulittle16_t GSSyms;
> >
> >
> > _______________________________________________
> > llvm-commits mailing list
> > llvm-commits at lists.llvm.org
> > http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160426/99348cea/attachment.html>