[PATCH] D153655: [LTO][GlobalDCE] Use pass parameter instead of module flag for LTO phase
Teresa Johnson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 23 14:37:10 PDT 2023
tejohnson added a comment.
In D153655#4445020 <https://reviews.llvm.org/D153655#4445020>, @aeubanks wrote:
> In D153655#4444985 <https://reviews.llvm.org/D153655#4444985>, @nikic wrote:
>
>> You need to also implement printing and parsing support for the new pass option.
>
> (https://reviews.llvm.org/D153437 as an example)
Ah missed that. Done.
I also updated new-pm-print-pipeline.ll to test the new parameter.
Expanded virtual-functions-visibility-post-lto.ll test to use globaldce with the new parameter in addition to testing by invoking the full regular LTO pass pipeline. I also expanded this to ensure the transformation doesn't occur if we don't specify the new parameter (or use a non-lto default pipeline).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153655/new/
https://reviews.llvm.org/D153655
More information about the llvm-commits
mailing list