[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