[llvm] r271137 - Use consume instead of manually using drop_front

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Sat May 28 12:17:49 PDT 2016


Author: majnemer
Date: Sat May 28 14:17:48 2016
New Revision: 271137

URL: http://llvm.org/viewvc/llvm-project?rev=271137&view=rev
Log:
Use consume instead of manually using drop_front

Modified:
    llvm/trunk/tools/llvm-readobj/COFFDumper.cpp

Modified: llvm/trunk/tools/llvm-readobj/COFFDumper.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-readobj/COFFDumper.cpp?rev=271137&r1=271136&r2=271137&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-readobj/COFFDumper.cpp (original)
+++ llvm/trunk/tools/llvm-readobj/COFFDumper.cpp Sat May 28 14:17:48 2016
@@ -998,10 +998,10 @@ void COFFDumper::mergeCodeViewTypes(Memo
     if (SectionName == ".debug$T") {
       StringRef Data;
       error(S.getContents(Data));
-      unsigned Magic = *reinterpret_cast<const ulittle32_t *>(Data.data());
+      uint32_t Magic;
+      error(consume(Data, Magic));
       if (Magic != 4)
         error(object_error::parse_failed);
-      Data = Data.drop_front(4);
       ArrayRef<uint8_t> Bytes(reinterpret_cast<const uint8_t *>(Data.data()),
                               Data.size());
       ByteStream Stream(Bytes);




More information about the llvm-commits mailing list