Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
This still requires some more tests to be added (which I am working on), and docs updated. I just wanted to get this out here for some early code reviews and testing on hw.
Paths
| Differential D59517
AMDGPU: Add support for cross address space synchronization scopes ClosedPublic Authored by kzhuravl on Mar 18 2019, 3:37 PM.
Details
Diff Detail Event TimelineHerald added subscribers: Petar.Avramovic, jfb, tpr and 6 others. · View Herald TranscriptMar 18 2019, 3:37 PM Comment Actions This still requires some more tests to be added (which I am working on), and docs updated. I just wanted to get this out here for some early code reviews and testing on hw. kzhuravl marked 2 inline comments as done. Comment ActionsAdded docs and tests. Addressed review feedback. This revision is now accepted and ready to land.Mar 19 2019, 1:40 PM Closed by commit rL356946: AMDGPU: Add support for cross address space synchronization scopes (authored by kzhuravl). · Explain WhyMar 25 2019, 1:49 PM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 191384 docs/AMDGPUUsage.rst
lib/Target/AMDGPU/AMDGPUMachineModuleInfo.h
lib/Target/AMDGPU/AMDGPUMachineModuleInfo.cpp
lib/Target/AMDGPU/SIMemoryLegalizer.cpp
test/CodeGen/AMDGPU/atomicrmw-nand.ll
test/CodeGen/AMDGPU/llvm.amdgcn.update.dpp.ll
test/CodeGen/AMDGPU/local-atomics-fp.ll
test/CodeGen/AMDGPU/memory-legalizer-amdpal.ll
test/CodeGen/AMDGPU/memory-legalizer-atomic-cmpxchg.ll
test/CodeGen/AMDGPU/memory-legalizer-atomic-fence.ll
test/CodeGen/AMDGPU/memory-legalizer-atomic-insert-end.mir
test/CodeGen/AMDGPU/memory-legalizer-atomic-rmw.ll
test/CodeGen/AMDGPU/memory-legalizer-invalid-addrspace.mir
test/CodeGen/AMDGPU/memory-legalizer-load.ll
test/CodeGen/AMDGPU/memory-legalizer-local.mir
test/CodeGen/AMDGPU/memory-legalizer-mesa3d.ll
test/CodeGen/AMDGPU/memory-legalizer-multiple-mem-operands-atomics.mir
test/CodeGen/AMDGPU/memory-legalizer-region.mir
test/CodeGen/AMDGPU/memory-legalizer-store.ll
|
Should expand name, "As" is not clear