[PATCH] D125404: [SPIR-V] Add simple tests to improve test coverage

Andrey Tretyakov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 16 14:09:41 PDT 2022


andreytr added inline comments.


================
Comment at: llvm/test/CodeGen/SPIRV/TruncToBool.ll:9-10
+; ModuleID = 'main'
+target datalayout = "e-i64:64-v16:16-v24:32-v32:32-v48:64-v96:128-v192:256-v256:256-v512:512-v1024:1024-n8:16:32:64"
+target triple = "spirv64-unknown-unknown"
+
----------------
arsenm wrote:
> Generally tests use -mtriple on the command line and don't repeat the triple/datalayout in the IR
Without "target triple = "spirv64-unknown-unknown" " it compiles x86 assembler.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D125404/new/

https://reviews.llvm.org/D125404



More information about the llvm-commits mailing list