[PATCH] D22141: [BFI] Add option to lazily calculate BFI

David Li via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 8 12:32:47 PDT 2016


davidxl added a comment.

There are already too many classes related to BFI, so I am not a fan of introducing another LazyBFI.

I suggest just making BFI lazy itself (when getBlockFreq() is first called ...). In fact  I think the laziness handling should be in BlocFrequencyInfoImpl class. By so doing, MBFI can get the laziness automatically as well.


http://reviews.llvm.org/D22141





More information about the llvm-commits mailing list