[PATCH] D146648: [MLIR][OpenMP] Added MLIR translation support for use_device clauses

Kiran Chandramohan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 06:05:38 PDT 2023


kiranchandramohan added a comment.

There is a build failure currently.



================
Comment at: mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp:1409
+    if (!mapOp.getType().isa<LLVM::LLVMPointerType>())
+      return failed();
 
----------------
failure()?


================
Comment at: mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp:1428
+    if (!devOp.getType().template isa<LLVM::LLVMPointerType>())
+      return failed();
+
----------------
failure()?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146648



More information about the llvm-commits mailing list