================ @@ -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