================
@@ -8130,17 +8135,19 @@ Function *OpenMPIRBuilder::emitUserDefinedMapper(
Value *OffloadingArgs[] = {MapperHandle, CurBaseArg, CurBeginArg,
CurSizeArg, CurMapType, CurNameArg};
- Function *ChildMapperFn = nullptr;
- if (CustomMapperCB && CustomMapperCB(I, &ChildMapperFn)) {
+
+ auto ChildMapperFn = CustomMapperCB(I);
+ if (!ChildMapperFn)
+ return ChildMapperFn.takeError();
+ if (*ChildMapperFn)
// Call the corresponding mapper function.
- Builder.CreateCall(ChildMapperFn, OffloadingArgs)->setDoesNotThrow();
- } else {
+ Builder.CreateCall(*ChildMapperFn, OffloadingArgs)->setDoesNotThrow();
+ else
----------------
skatrak wrote:
Nit: We need braces here because of the comments inside both branches.
https://github.com/llvm/llvm-project/pull/124746
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
