[PATCH] D64006: [Support] Support 64-bit offsets in DataExtractor.

[Igor Kudrin via Phabricator via llvm-commits]

ikudrin marked an inline comment as done.
ikudrin added inline comments.


================
Comment at: include/llvm/Support/DataExtractor.h:82
   ///     NULL will be returned.
+  const char *getCStr(uint64_t *offset_ptr) const;
+
----------------
aprantl wrote:
> Why do we need both variants? Would the 64-bit variant alone be sufficient?
In theory, yes. But the class is widely used in LLVM, and not only to parse DWARF sections. Trying to switch to `uint64_t` everywhere will result in a huge patch which will be hard to review. Hence, I decided to move gradually.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D64006/new/

https://reviews.llvm.org/D64006