[PATCH] D153349: [AMDGPU] Preserve dom-tree analysis in atomic optimizer.

Wed Jun 21 03:45:24 PDT 2023

arsenm accepted this revision.
arsenm added a comment.

In D153349#4437262 <https://reviews.llvm.org/D153349#4437262>, @pravinjagtap wrote:

> In D153349#4437207 <https://reviews.llvm.org/D153349#4437207>, @arsenm wrote:
>
>> Why doesn’t this need an llc-pipeline or opt-pipeline update? Do we not have new PM pass testing?
>
> Atomic optimizer pass is not configured with opt and llc uses legacy PM which already preserve this analysis. Therefore opt/llc pipeline doesn't need update. I think, There is no test for NewPM with pre-codegen (IR) passes.

We should have one. I thought we had an opt pipeline test but I don't see one, other targets do

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153349