[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]

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