[PATCH] D63676: Disable hosting MI to hotter basic blocks

Victor Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 12 19:16:42 PDT 2019


NeHuang marked 5 inline comments as done.
NeHuang added inline comments.


================
Comment at: llvm/lib/CodeGen/MachineLICM.cpp:92
+                cl::values(clEnumValN(Disable, "disable",
+			   "disable the feature"),
+                           clEnumValN(Enable_PGO, "enable-pgo",
----------------
lei wrote:
> indentation
Done 


================
Comment at: llvm/lib/CodeGen/MachineLICM.cpp:111
+STATISTIC(NumNotHoistedDueToHotness,
+	  "Number of instructions not hoisted due to block frequency hotness");
 
----------------
lei wrote:
> nit: please follow the same indentation as the lines above.
Done


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63676/new/

https://reviews.llvm.org/D63676





More information about the llvm-commits mailing list