[PATCH] D66985: [llvm-profdata] Recover from InstrProf read failures

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 10:01:01 PDT 2019


vsk added a comment.

In D66985#1655761 <https://reviews.llvm.org/D66985#1655761>, @davidxl wrote:

> yes, the old behavior should be the default. It raises the awareness of the problem (and merge errors are usually rare in reality).


Right, that's a good point. I wonder if it's worth introducing a separate mode, then, if it's not the recommended way to run the tool in production? Perhaps this should be set aside.


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

https://reviews.llvm.org/D66985





More information about the llvm-commits mailing list