[PATCH] D34418: [DWARF] Support for DW_FORM_strx3 and completed support for DW_FORM_strx{1, 2, 4}/consumer

Wolfgang Pieb via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 17:51:06 PDT 2017


wolfgangp added inline comments.


================
Comment at: lib/Support/DataExtractor.cpp:80
+  uint32_t Val = ExtractedVal.Bytes[0] + (ExtractedVal.Bytes[1] << 8) +
+                 (ExtractedVal.Bytes[2] << 16);
+  return Val;
----------------
aprantl wrote:
> perhaps make this a member UInt24_t::getAsUint32(LE?)() ?
Thanks, yes, that looks cleaner.


https://reviews.llvm.org/D34418





More information about the llvm-commits mailing list