[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