[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