[PATCH] D20184: [BranchFolding] Replace MachineBlockFrequencyInfo with MBFIWrapper. NFC.
Haicheng Wu via llvm-commits
llvm-commits at lists.llvm.org
Wed May 25 08:50:29 PDT 2016
haicheng added inline comments.
================
Comment at: lib/CodeGen/BranchFolding.h:29
@@ +28,3 @@
+ /// blocks and tail-merged blocks.
+ class MBFIWrapper {
+ public:
----------------
mssimpso wrote:
> Do you need to take this class out of the BranchFolder scope or would a forward declaration work?
Forward declaration works. At first I just wanted to type less letters when this class is used outside of BranchFolder :)
Repository:
rL LLVM
http://reviews.llvm.org/D20184
More information about the llvm-commits
mailing list