[PATCH] D112184: [test][Attributor] Remove legacy PM RUN lines

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added a comment.

In D112184#3076661 <https://reviews.llvm.org/D112184#3076661>, @jdoerfert wrote:

> In D112184#3076635 <https://reviews.llvm.org/D112184#3076635>, @aeubanks wrote:
>
>> `createAttributorLegacyPass()` and `createAttributorCGSCCLegacyPass()` are only used in PassManagerBuilder, which is the legacy PM's optimization pipeline (not codegen pipeline), which is deprecated
>
> AMDGPU backend creates an Attributor instance explicitly: `llvm/lib/Target/AMDGPU/AMDGPUAttributor.cpp`

Ah I see. Would it make sense to move this into the optimization pipeline? e.g. add it into `AMDGPUTargetMachine::registerPassBuilderCallbacks()`, probably as a `PB.registerOptimizerLastEPCallback()`.
+ at arsenm


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112184