[Openmp-commits] [PATCH] D60884: [OpenMP][libomptarget] Enable usage of unified memory for declare target link variables

George Rokos via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Jun 3 11:31:06 PDT 2019

grokos added inline comments.

Comment at: libomptarget/plugins/exports:5
+    __tgt_rtl_init_requires;
gtbercea wrote:
> grokos wrote:
> > One last question because I'm not very familiar with exporting symbols. If a plugin does not implement `__tgt_rtl_init_requires` (as is the case with generic-elf-64), will there be any problem with exporting the symbol?
> Supporting that method is optional so the function will not get called in that case. In the next chunk of added code you can see that the call to the init_requires() function is guarded:
> ```
> if (RTL->init_requires)
>     RTL->init_requires(RTLRequiresFlags);
> ```
My question was different. I meant, while building `libomptarget.rtl.x86_64.so` the linker is instructed to export a specific symbol from the exports list but this symbol is not defined in that plugin. Will the linker complain about the missing symbol or just ignore it? I suspect it's the latter, but I just want to make sure.

  rOMP OpenMP



More information about the Openmp-commits mailing list