[PATCH] D25583: Timer: Track Name and Description.

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 13 17:44:38 PDT 2016


MatzeB added a comment.

In https://reviews.llvm.org/D25583#569843, @vsk wrote:

> Could you either delete Timer::getName(), or audit it / upgrade its uses? There is no coverage for it either, so it can probably go along with getNamedRegionTimer.


Good point. Removing it produces no compile error here, so I think it is indeed unused. However being the only accessor function to this information I'd rather not delete it.


Repository:
  rL LLVM

https://reviews.llvm.org/D25583





More information about the llvm-commits mailing list