[PATCH] D58170: [DTU] Refine the interface and logic of applyUpdates

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 20 10:25:11 PST 2019


kuhar added a comment.

I think the default should be the `strict` mode; any code that relies on imprecise information about updates should make it clear by using some longer name, like `applyUpdatesRelaxed`.


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

https://reviews.llvm.org/D58170





More information about the llvm-commits mailing list