[PATCH] D101211: [ADT] Remove StatisticBase and make NoopStatistic empty

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 26 13:46:26 PDT 2021


MaskRay added a comment.

In D101211#2717753 <https://reviews.llvm.org/D101211#2717753>, @dblaikie wrote:

> "has 3 unused pointers" is the bit I had in mind in terms of modifying the description - that statement seems incorrect to me (in the presence of the LoopFuse usage). So I was thinking something like "has 3 mostly unused pointers" and then down at the end
>
> "the one use of these pointers is in LoopFuse (an unused pass) for printing out an optimization remark - for now that usage has been #ifdef'd out, if it's desirable to have the remark, it could use a different abstraction to keep track of the strings, or these particular statistics could be hardcoded to TrackingStatistic"

Oh, did not see this when I pushed the commit (I had a script running the last round of testing before pushing).

Yes, `has 3 mostly unused pointers` would be more accurate

> "the one use of these pointers is in LoopFuse (an unused pass) for printing out an optimization remark - for now that usage has been #ifdef'd out, if it's desirable to have the remark, it could use a different abstraction to keep track of the strings, or these particular statistics could be hardcoded to TrackingStatistic"

This description would be more accurate as well:)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101211



More information about the llvm-commits mailing list