[clang] Patch series to reapply #118734 and substantially improve it (PR #120534)

[Eli Friedman via cfe-commits]

================
@@ -100,10 +244,17 @@ class Context {
 
   /// Return the identifier name for the specified builtin,
   /// e.g. "__builtin_abs".
-  llvm::StringRef getName(unsigned ID) const { return getRecord(ID).Name; }
+  std::string getName(unsigned ID) const;
+
+  /// Return the identifier name for the specified builtin inside single quotes
+  /// for a diagnostic, e.g. "'__builtin_abs'".
+  std::string getQuotedName(unsigned ID) const;
----------------
efriedma-quic wrote:

Can you land getQuotedName() separately?

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