[PATCH] D49493: [DebugInfo] Reduce debug_str_offsets section size

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 19 09:14:52 PDT 2018


probinson added a comment.

I agree the ordering test looks fragile.  The only other option that comes to mind is trying to exercise the APIs directly from a unittest, but then you need enough scaffolding around it to capture at least the assembler output and verify it.  Probably best to leave the test as it is.


Repository:
  rL LLVM

https://reviews.llvm.org/D49493





More information about the llvm-commits mailing list