[SYCL][Driver] Enable SPV_INTEL_memory_access_aliasing extension

[maarquitos14]

No description provided.

[mdtoguchi]

@maarquitos14, will there be additional changes done to the clang-linker-wrapper to add this extension for the new offloading model?

[maarquitos14]

@maarquitos14, will there be additional changes done to the clang-linker-wrapper to add this extension for the new offloading model?

Tbh, I just adapted #4025 to the current codebase. What additional changes do we need to add this extension for the new offloading model?

[mdtoguchi]

@maarquitos14, will there be additional changes done to the clang-linker-wrapper to add this extension for the new offloading model?

Tbh, I just adapted #4025 to the current codebase. What additional changes do we need to add this extension for the new offloading model?

Perform a similar addition to

llvm/clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp

Line 764 in 2422e85

getTripleBasedSPIRVTransOpts(const ArgList &Args,

should take care of it

[maarquitos14]

@maarquitos14, will there be additional changes done to the clang-linker-wrapper to add this extension for the new offloading model?

Tbh, I just adapted #4025 to the current codebase. What additional changes do we need to add this extension for the new offloading model?

Perform a similar addition to

llvm/clang/tools/clang-linker-wrapper/ClangLinkerWrapper.cpp

Line 764 in 2422e85

getTripleBasedSPIRVTransOpts(const ArgList &Args,

should take care of it

Done :)

[mdtoguchi]

Thanks - OK for driver. Is there some test for the extensions for clang-linker-wrapper?

[maarquitos14]

Thanks - OK for driver. Is there some test for the extensions for clang-linker-wrapper?

I checked, but didn't find any. Glad to update it if I missed it.

[AlexeySachkov]

Thanks - OK for driver. Is there some test for the extensions for clang-linker-wrapper?

I checked, but didn't find any. Glad to update it if I missed it.

@maksimsab, @asudarsa, do you know?

[maksimsab]

Thanks - OK for driver. Is there some test for the extensions for clang-linker-wrapper?

It looks like we are missing such a test.

[maarquitos14]

Thanks - OK for driver. Is there some test for the extensions for clang-linker-wrapper?

It looks like we are missing such a test.

@mdtoguchi @AlexeySachkov should I add it here or should that be addressed in a different PR?

[maarquitos14]

@mdtoguchi @AlexeySachkov ping

[AlexeySachkov]

Thanks - OK for driver. Is there some test for the extensions for clang-linker-wrapper?

It looks like we are missing such a test.

@mdtoguchi @AlexeySachkov should I add it here or should that be addressed in a different PR?

Fine by me to have it in a separate PR. Especially considering that longer term clang-linker-wrapper should not use the translator as a binary, but instead should use it through a library call

[maarquitos14]

@intel/llvm-gatekeepers I think this is ready to merge :)

[sarnex]

@maarquitos14 I'm pretty sure you're already working on this but this broke all invokesimd tests (~55 tests) on dev igc arc, so I'm going to revert this, discussed with Jinsong.