================
@@ -473,6 +473,15 @@ def OffloadModuleInterface :
OpInterface<"OffloadModuleInterface"> {
return ::llvm::dyn_cast<BoolAttr>(isTargetDevice).getValue();
return false;
}]>,
+ InterfaceMethod<
+ /*description=*/[{
+ Return true if the current module was generated with OpenMP enabled.
+ }],
+ /*retTy=*/"bool",
+ /*methodName=*/"getIsOpenMP",
+ (ins), [{}], [{
+ return $_op->hasAttr("omp.is_target_device");
+ }]>,
----------------
jsjodin wrote:
Is there a reason to have this optional? I guess it would affect tests, but
adding an extra attribute would do the same I thnk. This is an indirect way of
checking if OpenMP is used so it looks a bit odd, although having two
attributes is also not great since they can be inconsistent with each other if
something goes wrong.
https://github.com/llvm/llvm-project/pull/209592
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
