[llvm] [AMDGPU] Add amdgpu-lower-exec-sync pass to lower named-barrier globals (PR #165692)
Sameer Sahasrabuddhe via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 11 06:57:52 PST 2025
ssahasra wrote:
> For any pass that will be newly introduced in the amdgpu backend, developers usually follow the approach of two PRs
>
> 1. Creating the pass, with both legacy and new pass manager implementations.
>
> 2. Adding the pass into the `amdgpu` pipeline.
>
>
> Have followed the same for this change as well. Additionally, removal of changes from `lower-module-lds` were made into separate PR as per feedback from @shiltian
This is not a new pass. It's _existing code_ bundled into a pass as a refactoring. I still think this should be a single change from the point of view of someone looking at the git log. All failures should be traceable to this one transaction.
https://github.com/llvm/llvm-project/pull/165692
More information about the llvm-commits
mailing list