[PATCH] D134456: [PGO] Consider parent context when weighing branches with likelyhood.

Tue Sep 27 14:25:35 PDT 2022

rnk added a comment.

In D134456#3819042 <https://reviews.llvm.org/D134456#3819042>, @aaron.ballman wrote:

> Alternatively, we could document that we don't follow the intent of the standard feature and that's just the way things are, and add a command line option to honor that intent. However, given that GCC honors the attribute over PGO, I think we should change our default behavior.

That makes sense to me. I think it's reasonable to request folks to add a flag in the context of a bug fix, but it would be too much to ask a drive by contributor to go through the multi-step process to change the default flag behavior.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134456