[PATCH] D79920: [llvm][NFC] Simplify ProfileSummaryInfo state transitions.

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 20 14:18:41 PDT 2020


mtrofin added a comment.

In D79920#2047502 <https://reviews.llvm.org/D79920#2047502>, @yamauchi wrote:

> I still think it'd be best to have the pure formatting changes (clang-format) in a separate patch.


I think it's a matter of what scenarios we have: do we have a scenario for ever re-updating the PSI or not.

We do need to work that out, and that's my intention. In the meantime, I believe it would be helpful to simplify the current state of the code. As this patch clarifies state transitions, it should be relatively easy to further simplify it further afterwards.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79920





More information about the llvm-commits mailing list