[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
Tue Jul 25 02:56:19 PDT 2023
kiranchandramohan added a comment.
> BTW I am not sure how to resolve the windows build error that I am getting repeatedly. It's giving an unknown identifier error for the fail functor being called at OpenMPToLLVMIRTranslation.cpp:1429. Any insight would be much appreciated.
Might be a bug in the compiler in windows. You are trying to call a lambda function inside a lambda function here. Try moving the second lambda into a static function or try capturing the second lambda explicitly in the first lambda.
================
Comment at: mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp:1386
llvm::OpenMPIRBuilder *ompBuilder = moduleTranslation.getOpenMPBuilder();
unsigned index = 0;
+ auto fail = [&combinedInfo]() -> void {
----------------
Can you move this to just before the loop at 1406?
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