[SYCL] Fix macro in device library function declaration to distinguis…

[jinge90]

…h std libraries

Signed-off-by: gejin [email protected]

[romanovvlad]

@jinge90 What is the problem with current macro?

[jinge90]

@jinge90 What is the problem with current macro?

Hi, Vlad. As our compiler are based on LLVM , so:
"if defined(GNUC) || defined(clang)" will always be true as clang will always be defined
and "elif defined(_MSC_VER)" will always be false as only _MSC_VER is a Microsoft CL compiler's macro telling its version. Current code will lead to std::assert building failure on Windows.
Thank you very much.

[romanovvlad]

@jinge90 What is the problem with current macro?

Hi, Vlad. As our compiler are based on LLVM , so:
"if defined(GNUC) || defined(clang)" will always be true as clang will always be defined
and "elif defined(_MSC_VER)" will always be false as only _MSC_VER is a Microsoft CL compiler's macro telling its version. Current code will lead to std::assert building failure on Windows.
Thank you very much.

I see. I would like @asavonic to approve.