[PATCH] D62541: Adding a function for setting coverage output file.

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 29 16:43:42 PDT 2019


davidxl added a comment.

I think it is cleaner for user to make the intention clear by passing the flag if the file handle is passed to the runtime. Otherwise the runtime will have to guess/assume user's intention (e.g, also reopen the file with "r+b" mode for merging).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62541





More information about the llvm-commits mailing list