[PATCH] D67276: [DWARF] Add a unit test for DWARFUnit::getLength().

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 6 09:13:12 PDT 2019


probinson added a comment.

As written, this will work only for little-endian.  There are other unittests that are bi-endian that you ought to be able to use for examples; if nothing else, bail out after checking `sys::IsLittleEndianHost` (but a bi-endian test would be better).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D67276





More information about the llvm-commits mailing list