[Mlir-commits] [mlir] [mlir][nvvm] Remove unused check-ptx (PR #93147)

Guray Ozen llvmlistbot at llvm.org
Fri May 24 03:49:35 PDT 2024


================
@@ -6,15 +6,6 @@
 // RUN:   --entry-point-result=void \
 // RUN:  | FileCheck %s
 
-// Basic PTX check to make sure we are generating the right instructions.
-
-// CHECK-PTX: mbarrier.init.shared.b64
----------------
grypp wrote:

There are merits of checking the PTX assembly. But this is an integration test. It's not the best place the test PTX. I've added that check earlier, but we now test the generated PTX in the conversion tests. 

https://github.com/llvm/llvm-project/pull/93147


More information about the Mlir-commits mailing list