[llvm] [SYCL][LLVM] Adding property set I/O library for SYCL (PR #110771)
Joseph Huber via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 15 15:08:36 PDT 2024
jhuber6 wrote:
> 2\. If it is an implementation detail, can it be changed to a standard defined textual format (YAML, JSON, XML)? I ask this specifically because custom serialization formats are often great targets for security vulnerabilities, so using a standardized and defined format is preferable to making your own. Obviously if this is a required part of the SYCL specification that has different considerations.
My gut instinct is that this should probably be a binary format w/ all the YAML serialization all the others get. But my assumption is that SYCL's implementation uses this somehow so I'm unsure how much work it is to rip it all up and do something else.
https://github.com/llvm/llvm-project/pull/110771
More information about the llvm-commits
mailing list