|33 ↗||(On Diff #169197)|
The intent here is to make this look less like the typical "bring this name into the namespace" ane more like "give this a new name", even if the unqualified name is the same.
In particular: this no longer matches rg -t h "using llvm::" which is a good way to find such bad decls :-) It's not a big deal, but I think this is a marginal improment.
It's preferred to include clang/Basic/LLVM.h instead and use the StringRef in the clang namespace:
Otherwise we'd have llvm::StringRef literally everywhere.