[PATCH] D32073: [ProfileData] Support cross target binary reading for coverage tool

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 13 22:36:36 PDT 2017


davidxl added a comment.

The main client (the compiler) should not need to know the details so a higher level interface taking 'Module' should be better. It is the host tool that need to use the lower level interface. For  the later use, there was never a need for segment prefix, so adding the flexibility is not necessary.


https://reviews.llvm.org/D32073





More information about the llvm-commits mailing list