[PATCH] D32447: Update profile during memory instrinsic optimization

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 24 11:51:34 PDT 2017


davidxl added inline comments.


================
Comment at: lib/Transforms/Instrumentation/IndirectCallPromotion.cpp:955
   MergeBB->setName("MemOP.Merge");
+  // Set the block frequency on the MergeBB, since it contains the
+  // remainder of the original BB, and may contain additional mem ops
----------------
This comment seems unnecessary :) Also can you move the DefaultBB setting after MergeBB update?


https://reviews.llvm.org/D32447





More information about the llvm-commits mailing list