[PATCH] D126340: [clang][AIX] add option -mdefault-visibility-export-mapping

Thu Jun 2 05:40:19 PDT 2022

daltenty added a comment.

In D126340#3552645 <https://reviews.llvm.org/D126340#3552645>, @nikic wrote:

> It looks like this causes a major compile-time regression in the clang frontend: http://llvm-compile-time-tracker.com/compare.php?from=6232a8f3d61e5856c17e7b314385e9ea8068cdc1&to=8c8a2679a20f621994fa904bcfc68775e7345edc&stat=instructions For example, tramp3d-v4 is up 5% in `-O0` builds.
>
> Not familiar with this area, but my first guess would be that getLinkageAndVisibility() is actually expensive and you're introducing a ton of calls to it?

@nikic thanks for catching this! Looking into it now, I'll put in a revert if I can't get things sorted shortly

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126340