[PATCH] D125847: LTO: Decide upfront whether to use opaque/non-opaque pointer types

[Steven Wu via Phabricator via llvm-commits]

steven_wu added a comment.

In D125847#3528111 <https://reviews.llvm.org/D125847#3528111>, @aeubanks wrote:

> I'm still looking into a whole program devirtualization bug that only repros with opaque pointers, and there are still potential performance issues to look into

Thanks. Maybe here is another way to look at this problem, instead of giving people a temporary switch to use opaque_pointer if their first bitcode is no_opaque_pointer, we opt as aggressive as possible into opaque_pointer and give people a temporary switch to go back to old behavior just like how clang is configured.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125847