[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