search

KhronosGroup / SPIRV-LLVM-Translator

A tool and a library for bi-directional translation between SPIR-V and LLVM IR
Other
468 stars 209 forks source link

Don't add initializer to local variables with external linkage #2637

Closed maarquitos14 closed 1 month ago

maarquitos14 commented 2 months ago

If a variable is external, it will be defined during linking, the translator should not do it.

MrSidims commented 2 months ago

I agree with the change and will approve after adding a test.

But would like to hear from Sven.

maarquitos14 commented 1 month ago

I agree with the change and will approve after adding a test.

But would like to hear from Sven.

Test added.