[PATCH] D22744: CodeExtractor : Add ability to preserve profile data.
River Riddle via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 27 11:14:45 PDT 2016
rriddle added inline comments.
================
Comment at: lib/Transforms/Utils/CodeExtractor.cpp:693
@@ +692,3 @@
+ for (auto &Weight : ExitWeights) {
+ if (Weight.second.getFrequency() == 0) {
+ HasValidWeights = false;
----------------
davidxl wrote:
> This check does not look correct. It is possible for the frequency to be zero
Yeah, that was my understanding. Thanks for the sanity check! I need to add the weights without using the current methods given that they do not allow for weights of 0.
https://reviews.llvm.org/D22744
More information about the llvm-commits
mailing list