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

Steven Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 20 11:18:06 PDT 2022


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



More information about the llvm-commits mailing list