[PATCH] D153744: [LoopUnroll] adjust for new `convergent` semantics

Jon Chesterfield via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 26 05:16:41 PDT 2023


JonChesterfield added a comment.

> ! In D153744#4448146 <https://reviews.llvm.org/D153744#4448146>, @sameerds wrote:
>  As for documentation, there's definitely been a LOT more activity than just a "slide deck or two", both inside AMD as well as outside. Please do refer to the "stack" of reviews that this Phab review belongs to.
>
> Also, there's this, for links to all historical stuff:
>  https://discourse.llvm.org/t/rfc-introduce-convergence-control-intrinsics/69613

It's been a multi-year thing with multiple authors, I doubt trawling the history will provide anyone with an accurate idea of what we're currently trying to build.

This stack is revising how control flow is modelled in llvm for GPUs. The failure modes are substantial. Could part of the implementation effort be directed to writing down what the overall algorithm is intended to be, somewhere that is visible outside of AMD? Hopefully that's a direct adaption of whatever design document you are working from internally.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153744



More information about the llvm-commits mailing list