Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

[SYCL][Doc] Add test plan for dynamic linking feature #382

Merged
merged 1 commit into from
Jul 29, 2021

Conversation

Fznamznon
Copy link

No description provided.

@romanovvlad
Copy link

romanovvlad commented Jul 29, 2021

Do we support app(kernel) + dlopen + lib(SYCL_EXTERNAL) case, where kernel deps on SYCL_EXTERNAL from lib?
Sorry if missed this case in the doc.

@AlexeySachkov
Copy link

Do we support app(kernel) + dlopen + lib(SYCL_EXTERNAL) case, where kernel deps on SYCL_EXTERNAL from lib?
Sorry if missed this case in the doc.

Could you please clarify the use-case? Dependency on SYCL_EXTERNAL means that library must be linked as well in order for host part of the app to be linked without errors and if we have linked the library, what is the reason for dynamically loading it again?

@romanovvlad
Copy link

romanovvlad commented Jul 29, 2021

Do we support app(kernel) + dlopen + lib(SYCL_EXTERNAL) case, where kernel deps on SYCL_EXTERNAL from lib?
Sorry if missed this case in the doc.

Could you please clarify the use-case? Dependency on SYCL_EXTERNAL means that library must be linked as well in order for host part of the app to be linked without errors and if we have linked the library, what is the reason for dynamically loading it again?

From you answer I infer that this case is not supported. But still try to clarify:

  1. libA defines SYCL_EXTERNAL function foo
  2. An app has a kernel which depends on the foo defined in libA
  3. The app dlopen's libA (libA is not linked during compilation to the app!!)
  4. The app runs the kernel

@Fznamznon
Copy link
Author

@romanovvlad

The app dlopen's libA (libA is not linked during compilation to the app!!)

This is not supported by regular C++. There will be a link error.

@romanovvlad romanovvlad merged commit 204b096 into intel:intel Jul 29, 2021
aelovikov-intel pushed a commit to aelovikov-intel/llvm that referenced this pull request Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants