erichkeane wrote: Hmm... the quotes make it less obvious that these aren't supposed to be quoted in the Builtins.def file, so I don't know if we should do that. I think I'd rather we make it clear that `N` is sometimes. IF we had a way of formatting this instead of comments, perhaps highlighting this would be beneficial.
ONE thought I just thought of: what if instead of THIS, we extracted this Builtins.def file (which right now is a worthless/empty documentation-only file) into the `InternalsManual.rst`, and just left a 'link' to it here instead? That way we could use ITS 'this is code' type quoting to make it clear that we mean that. Additionally, we could include examples for each to make it more clear what they are doing. WDYT? https://github.com/llvm/llvm-project/pull/160080 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
