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

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 1 10:06:12 PDT 2019


aprantl added inline comments.


================
Comment at: include/llvm/Support/DataExtractor.h:82
   ///     NULL will be returned.
+  const char *getCStr(uint64_t *offset_ptr) const;
+
----------------
Why do we need both variants? Would the 64-bit variant alone be sufficient?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D64006





More information about the llvm-commits mailing list