https://github.com/efriedma-quic commented:

I'd like to consider future extensions a little more before we go forward with 
this.

Discussion #67698 suggests we might want to construct a "symbol table" for 
inline asm, and store it in the IR.  (I guess this actually fixes #67698 in a 
different way, though?)

#112920 might require attaching options related to include paths to the inline 
asm.

For [LTO with linker 
scripts](https://discourse.llvm.org/t/rfc-lto-lld-lto-with-linker-scripts-implemented/91089),
 we might want to attach origin filename, so we can emit that into the emitted 
object file.

We don't need to implement all of that immediately, of course, but maybe it 
makes sense to allow more generalized "metadata" so we don't need to hack core 
IR datastructures every time we want to extend the list of properties?

https://github.com/llvm/llvm-project/pull/204548
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to