[Openmp-commits] [openmp] [OpenMP][libomptarget] Enable automatic unified shared memory executi… (PR #75999)
Johannes Doerfert via Openmp-commits
openmp-commits at lists.llvm.org
Tue Dec 19 19:47:24 PST 2023
================
@@ -2913,7 +2913,7 @@ struct AMDGPUPluginTy final : public GenericPluginTy {
/// Create an AMDGPU plugin and initialize the AMDGPU driver.
AMDGPUPluginTy()
: GenericPluginTy(getTripleArch()), Initialized(false),
- HostDevice(nullptr) {}
+ HostDevice(nullptr), IsAPU(false), OMPX_HSAXnack("HSA_XNACK", false) {}
----------------
jdoerfert wrote:
(I wrote this before I saw Shilei's comment)
I understand you followed existing code but I think we should (maybe in a pre-commit) move these initializations to the member definition. It's cumbersome to have them here, in order, as one needs to look at 2 places to understand type and initial value.
https://github.com/llvm/llvm-project/pull/75999
More information about the Openmp-commits
mailing list