================
@@ -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;
----------------
chandlerc wrote:

Eh, it was quite easy to search for all uses and audit them. Plus, Clang 
actually caught a bunch with warnings for me.

But sure, I've renamed it to `getFullName` since we might choose to surface the 
prefix distinction in the future.

https://github.com/llvm/llvm-project/pull/120534
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to