[llvm] [SPIR-V] Expose an API call to initialize SPIRV target and translate input LLVM IR module to SPIR-V (PR #107216)
Nathan Gauër via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 10 05:29:00 PDT 2024
Keenuts wrote:
> What is your opinion about the change, does it look slightly better now?
If back/forward-compatibility is the most important part, I'd prefer to either:
- go full parsing (as you did before), losing compile-time checks and self-documenting code, but simple to use.
- do it like Vulkan (linked list of structs, head passed by pointer), bringing more compile-time checks/documentations, but making usage slightly more verbose.
I feel like having part in the to-parse command-line, and one part as classic parameter is weird.
https://github.com/llvm/llvm-project/pull/107216
More information about the llvm-commits
mailing list