[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