[llvm] [SPIRV] Emitting DebugSource, DebugCompileUnit (PR #97558)

via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 12 04:23:35 PDT 2024


bwlodarcz wrote:

@michalpaszkowski I'll add the option. I want to point out that there are two (I think) major negative consequences of that move:
a) It means that in the time between initial implementation of the DebugInfo and finishing it all other tests (for semantic code) won't be testing interaction with non semantic at all.  Basically it's like a disabling all tests for non semantic with the exception of it's own.
b) It's breaking user expectation. Let's imagine typical user who want to generate some SPIR-V. He is compiling some opencl code with clang with `-g` option and his reasonable expectation is to get DI valid for format which is conformant with the standard - even partial is better than none (IMO). He doesn't get it and now he needs to go through documentation/or worse code to find the name of the option and enable it.


https://github.com/llvm/llvm-project/pull/97558


More information about the llvm-commits mailing list