[PATCH] D152649: [AMDGPU] Enable Atomic Optimizer and Default to Iterative Scan Strategy.

Sun Jun 18 15:38:39 PDT 2023

jhuber6 added a subscriber: ye-luo.
jhuber6 added a comment.

In D152649#4431278 <https://reviews.llvm.org/D152649#4431278>, @b-sumner wrote:

> In D152649#4431258 <https://reviews.llvm.org/D152649#4431258>, @jhuber6 wrote:
>
>> @jplehr @ronlieb
>>
>> This patch seems to have had catastrophic results on the build times in the `libc` project. See the buildbot where the build times have gone from ~15 minutes to over an hour https://lab.llvm.org/staging/#/builders/247. Reverting this patch causes the time for my to build a test to go from 55 seconds to 5 seconds. I can try to get pass timing later.
>
> If this is only affecting AMDGPU and libc, is it a problem?  This optimization is very important.  Of course, it would be great if it can be sped up.

I don't know if it's just a `libc` problem, it probably just highlights the problem since it's a considerably large application with a good number of atomic operations thanks to RPC calls. Maybe @ye-luo can let me know if he's had any compile time regressions recently. In any case a ~10x increase in link time is a little steep for a single optimization. I can always disable this pass in the `libc` build but I think that other large applications are going to see similar changes so we need to think of a way to do this that isn't so slow.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152649