[llvm] [NVPTX] Add Volta Atomic SequentiallyConsistent Load and Store Operations (PR #98551)
Artem Belevich via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 25 15:42:43 PDT 2024
================
@@ -1,169 +1,7 @@
; RUN: llc < %s -march=nvptx64 -mcpu=sm_70 -mattr=+ptx82 | FileCheck %s
; RUN: %if ptxas-12.2 %{ llc < %s -march=nvptx64 -mcpu=sm_70 -mattr=+ptx82 | %ptxas-verify -arch=sm_70 %}
-; CHECK-LABEL: generic_plain
----------------
Artem-B wrote:
Why were these removed? I think it still makes sense to make sure that plain non-atomic loads/stores still work as expected. It may be redundant with other load/store tests. In that case please add a comment 'plain loads/stores without atomic modifiers are tested in ....'
Also, if the file is now testing only atomic loads/stores, it would make sense to rename it accordingly. Then the comment about non-atomic variants will not be necessary.
https://github.com/llvm/llvm-project/pull/98551
More information about the llvm-commits
mailing list