[PATCH] D111322: Remove unnecessary StringRef convesion in llvm-config

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 8 15:33:47 PDT 2021


craig.topper added a comment.

I think this still ends up doing an implicit conversion to StringRef first since there is no SmallString constructor from const char *. But I guess shorter code is better.

LGTM


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111322



More information about the llvm-commits mailing list