[SYCL] Fix LIT tests to take into account LLVM_LIBDIR_SUFFIX

[fwyzard]

sycl/test/CMakeLists.txt uses

set(LLVM_BUILD_LIBRARY_DIRS "${LLVM_BINARY_DIR}/lib/")

instead of

set(LLVM_BUILD_LIBRARY_DIRS "${LLVM_BINARY_DIR}/lib${LLVM_LIBDIR_SUFFIX}")

making all LIT tests will fail if a LLVM_LIBDIR_SUFFIX is specified.

This PR updates the path to take LLVM_LIBDIR_SUFFIX into account.

[bader]

@fwyzard, could you check if #1165 fixes the problem, please?
It allows to validate "deployed" compiler and it's already reviewed and approved for commit.

[fwyzard]

@fwyzard, could you check if #1165 fixes the problem, please?

I think it misses ${LLVM_LIBDIR_SUFFIX} as well.

It allows to validate "deployed" compiler and it's already reviewed and approved for commit.

Does it mean it allows to validate only a deployed compiler, or also a deployed compiler ?

In fact, how does the deployment work ?
The instructions at GetStartedWithSYCLCompiler.md say only

TODO: add instructions how to deploy built SYCL toolchain.

[bader]

It allows to validate "deployed" compiler and it's already reviewed and approved for commit.

Does it mean it allows to validate only a deployed compiler, or also a deployed compiler ?

It allows to validate both: built and deployed compilers.

In fact, how does the deployment work ?
The instructions at GetStartedWithSYCLCompiler.md say only

TODO: add instructions how to deploy built SYCL toolchain.

There is a special cmake target deploy-sycl-toolchain, which is supposed to prepare a compiler toolchain with SYCL support.
Thanks for noticing missing instructions. I'll update the document later.

[fwyzard]

@fwyzard, could you check if #1165 fixes the problem, please?

I think it misses ${LLVM_LIBDIR_SUFFIX} as well.

Seems to work with #1165 (comment) .

[fwyzard]

#1165 together with #1165 (comment) should already fix this issue.