[SYCL] missing SYCL_EXPORT leading to downstream troubles (#7318)

cperkinsintel · web-flow · commit d82412764b4f · 2022-11-08T13:47:21.000-08:00
my earlier post-commit fix caused an ABI symbols problem downstream. But
the original problem is that the custom specialization of
`get_info&lt;parent_device&gt;` is merely missing an export declaration. If
I'm right, or just dumb lucky, then this will fix all downstream
failures, including post-commit. And if not, well, the symbol update is
correct regardless

Signed-off-by: Chris Perkins &lt;chris.perkins@intel.com&gt;
diff --git a/sycl/source/device.cpp b/sycl/source/device.cpp
@@ -128,7 +128,8 @@ device::get_info() const {
 }
 
 // Explicit override. Not fulfilled by #include device_traits.def below.
-template <> device device::get_info<info::device::parent_device>() const {
+template <>
+__SYCL_EXPORT device device::get_info<info::device::parent_device>() const {
   // With ONEAPI_DEVICE_SELECTOR the impl.MRootDevice is preset and may be
   // overridden (ie it may be nullptr on a sub-device) The PI of the sub-devices
   // have parents, but we don't want to return them. They must pretend to be

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,8 @@ device::get_info() const {`
`128`	`128`	`}`
`129`	`129`
`130`	`130`	`// Explicit override. Not fulfilled by #include device_traits.def below.`
`131`		`-template <> device device::get_info<info::device::parent_device>() const {`
	`131`	`+template <>`
	`132`	`+__SYCL_EXPORT device device::get_info<info::device::parent_device>() const {`
`132`	`133`	`// With ONEAPI_DEVICE_SELECTOR the impl.MRootDevice is preset and may be`
`133`	`134`	`// overridden (ie it may be nullptr on a sub-device) The PI of the sub-devices`
`134`	`135`	`// have parents, but we don't want to return them. They must pretend to be`