[PATCH] D139881: [clang] Use a StringRef instead of a raw char pointer to store builtin and call information

[Balázs Benics via Phabricator via llvm-commits]

steakhal added a comment.

The `StaticAnalyzer` portion looks good to me AFAICT.



================
Comment at: clang/lib/StaticAnalyzer/Core/CallDescription.cpp:39
 ento::CallDescription::CallDescription(CallDescriptionFlags Flags,
-                                       ArrayRef<const char *> QualifiedName,
+                                       ArrayRef<StringRef> QualifiedName,
                                        MaybeCount RequiredArgs /*= None*/,
----------------
Maybe we could restrict it even more to `StringLiteral`. The same applies to the other ctor.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139881/new/

https://reviews.llvm.org/D139881