[flang-commits] [PATCH] D121171: [flang] Add ExternalNameConversionPass to flang-new pipeline

[Valentin Clement via Phabricator via flang-commits]

clementval added a comment.

In D121171#3367315 <https://reviews.llvm.org/D121171#3367315>, @awarzynski wrote:

> There's one more alternative. `ExternalNameConversionPass` has a rather huge and visible impact on the generated output. It makes sense to have it enabled by default, but you could also add a flag (e.g. `-finternal-mangling` or some other spelling) to disable it. This way:
>
> - you'd only have to update  RUN lines in tests, and
> - we'd have a "compatibility" mode for `bbc` and `flang-new`, which IMO would be a good thing for reproducibility.

+1


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121171