================
@@ -30,7 +30,7 @@ std::string platformString() {
   return PlatformString;
 }
 
-std::string featureString() {
+llvm::StringRef featureString() {
----------------
ArcsinX wrote:

> But it strange for me why all the checks including the building and testing 
> did not fail here...

Remote index functionality is not enabled by default (and also has gRPC as 
additional requirement).
I guess that the number of configurations that are built and tested is limited 
for PRs, but after the merge into main, I think some build bots will be unhappy.

https://github.com/llvm/llvm-project/pull/180335
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to