[PATCH] D49046: Use StringRef instead of `const char *`.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 9 01:34:58 PDT 2018


grimar accepted this revision.
grimar added a comment.
This revision is now accepted and ready to land.

Looks reasonable, LGTM.

For LLD this seems helps to avoid double find of the name field length:
https://github.com/llvm-mirror/lld/blob/master/ELF/SyntheticSections.cpp#L2219


https://reviews.llvm.org/D49046





More information about the llvm-commits mailing list