[PATCH] D122450: [ELF] Default to --no-fortran-common

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 28 13:48:52 PDT 2022


MaskRay added a comment.

In D122450#3412334 <https://reviews.llvm.org/D122450#3412334>, @ikudrin wrote:

> If I understand it right, `--fortran-common` is useful in rare scenarios but can slow down linking when handling COMMON symbols, and it also hinders implementing the parallel symbol resolution. If that is true, I agree with the change.

The understanding is right.

If we implement parallel symbol resolution, we probably have two code paths for a while. Not wasting efforts on implementing --fortran-common for the parallel code path helps.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122450



More information about the llvm-commits mailing list