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

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 13:27:12 PDT 2016


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


More information about the llvm-commits mailing list