[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