[PATCH] D79396: [BrachProbablityInfo] Set edge probabilities at once. NFC.
Evgeniy via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 7 23:57:34 PDT 2020
ebrevnov added a comment.
I think the new API of setting probabilities for all edges "atomically" is a good move. There is one suggestion though. In fact, there is no need to "repack" vector of probabilities into map. We can simply copy or even move (in most cases we have rvalue) entire vector into a map. This can be done as a separate change though.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79396/new/
https://reviews.llvm.org/D79396
More information about the llvm-commits
mailing list