[libc] [compiler-rt] [clang] [libcxx] [llvm] [lldb] [flang] [clang-tools-extra] ✨ [Sema, Lex, Parse] Preprocessor embed in C and C++ (and Obj-C and Obj-C++ by-proxy) (PR #68620)

via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 28 08:55:28 PST 2023


cor3ntin wrote:

> So it's not so much that it's not actually true in the implementation details, it's that it's not actually true by specification either.

+1. I really do not want `-E` to start not expanding, or transforming directives.
I do not see a difference between a large embed and a large include file. `-E` should expand both, regardless of size. Modify that very long standing behavior only cause pain and confusion imo.

If, we find the need for a fancier behavior (like the aforementioned `#embed_base64`), it's something we could consider later, under an additional switch (not `-E` without other flag)

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


More information about the llvm-commits mailing list