[Mlir-commits] [mlir] [mlir][gpu] Add `subgroup_broadcast` op (PR #152808)
Jianhui Li
llvmlistbot at llvm.org
Wed Sep 3 18:16:56 PDT 2025
Jianhui-Li wrote:
> Hi @Jianhui-Li and @fabianmcg,
>
> I gave this a green light to merge because the comments seemed reasonably addressed from what I could tell and @Jianhui-Li did not leave a blocking review -- this is the tool to indicate that you do not wish the PR be merged as is.
>
> Since there's still a little bit to work out, I propose we do not revert the whole thing but instead either fix forward or drop just the `any_lane` variant, which I think is the only thing we haven't aggreged on. Does this sound reasonable @Jianhui-Li and @fabianmcg ?
I agree. Like I stated earlier, I agree with the `first_active_lane` and `specific_lane` variants but think more discussion needed to use `any_lane` as a compiler hint.
https://github.com/llvm/llvm-project/pull/152808
More information about the Mlir-commits
mailing list