[PATCH] D30119: MachineRegionInfo: fix pass initialization

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 17 15:11:25 PST 2017


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

LGTM with name fix



================
Comment at: lib/CodeGen/MachineRegionInfo.cpp:7
 
-#define DEBUG_TYPE "region"
+#define DEBUG_TYPE "machineregions"
 
----------------
machine-region-info to better match name


================
Comment at: lib/CodeGen/MachineRegionInfo.cpp:127
 
-INITIALIZE_PASS_BEGIN(MachineRegionInfoPass, "regions",
+INITIALIZE_PASS_BEGIN(MachineRegionInfoPass, "machineregions",
                 "Detect single entry single exit regions", true, true)
----------------
Use DEBUG_TYPE


================
Comment at: lib/CodeGen/MachineRegionInfo.cpp:132
 INITIALIZE_PASS_DEPENDENCY(MachineDominanceFrontier)
-INITIALIZE_PASS_END(MachineRegionInfoPass, "regions",
+INITIALIZE_PASS_END(MachineRegionInfoPass, "machineregions",
                 "Detect single entry single exit regions", true, true)
----------------
Ditto


https://reviews.llvm.org/D30119





More information about the llvm-commits mailing list