[SYCL] Store the Adapter as weak_ptr in build results

[slawekptak]

No description provided.

[vinser52]

Does this change impact the performance?

[slawekptak]

Does this change impact the performance?

This change should reduce the shared_ptr copies on the fast path by 2, right? I haven't measured this one, but this is one of the changes in the series of shared_ptr optimizations, so I would expect that overall the performance should be better.

[vinser52]

Does this change impact the performance?

This change should reduce the shared_ptr copies on the fast path by 2, right? I haven't measured this one, but this is one of the changes in the series of shared_ptr optimizations, so I would expect that overall the performance should be better.

Thank you! That was exactly my question: if this change impacts the hot path.

[KseniyaTikhomirova]

LGTM

[slawekptak]

Does this change impact the performance?

This change should reduce the shared_ptr copies on the fast path by 2, right? I haven't measured this one, but this is one of the changes in the series of shared_ptr optimizations, so I would expect that overall the performance should be better.

Thank you! That was exactly my question: if this change impacts the hot path.

Just to clarify - this affects the cache insertion flow, so once in a stable state, it seems like it doesn't affect the fast path.