[clang] Switch builtin strings to use string tables (PR #118734)

[Craig Topper via cfe-commits]

================
@@ -41,6 +41,10 @@ class LLVM_LIBRARY_VISIBILITY ARCTargetInfo : public TargetInfo {
                         MacroBuilder &Builder) const override;
 
   ArrayRef<Builtin::Info> getTargetBuiltins() const override { return {}; }
----------------
topperc wrote:

Should this override of getTargetBuiltins be removed? It looks like it matches the default.

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