MatzeB added a comment.

And as another strawman / discussion-started I put up D158680 
<https://reviews.llvm.org/D158680> where I use `!{"branch_weights", i32 1, i32 
0}` to represent likely branches and the actual "LikelyWeight" mostly becomes 
an internal implementation detail of the BranchProbabilityInfo class... This 
seemed like a neat way to get an abstract "likely", "unlikely" notation, but 
not sure of the effects if we no longer would have "truly zero" weights because 
they would be interpreted differently now...


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158668/new/

https://reviews.llvm.org/D158668

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to