[PATCH] D120544: [AMDGPU] Omit unnecessary waitcnt before barriers

Marek Olšák via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 2 15:00:08 PDT 2022


mareko added a comment.

In D120544#3486402 <https://reviews.llvm.org/D120544#3486402>, @nhaehnle wrote:

> In D120544#3466638 <https://reviews.llvm.org/D120544#3466638>, @kerbowa wrote:
>
>> I believe HIP adds a release fence before the barrier and an acquire directly after. Would something like that make sense?
>
> Yes, generally I expect that is what's required.

What does the release fence add? Waiting for only LDS or everything?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120544



More information about the llvm-commits mailing list