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

Wed Jun 21 05:05:43 PDT 2023

foad added a comment.

In D153349#4437321 <https://reviews.llvm.org/D153349#4437321>, @arsenm wrote:

> 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

See comments on https://github.com/llvm/llvm-project/commit/6f73bd781305266a747055875ce8352e5a36c809

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153349