[PATCH] D117644: [ELF] Remove StringRefZ

[Alexander Shaposhnikov via Phabricator via llvm-commits]

alexander-shaposhnikov added inline comments.


================
Comment at: lld/ELF/Symbols.h:250
          uint8_t stOther, uint8_t type)
-      : file(file), nameData(name.data), nameSize(name.size), binding(binding),
-        type(type), stOther(stOther), symbolKind(k), visibility(stOther & 3),
+      : file(file), nameData(name.data()), nameSize(name.size()),
+        binding(binding), type(type), stOther(stOther), symbolKind(k),
----------------
it looks like in both cases this code tries to "unpack" a StringRef into nameData and nameSize 
(here and inside setName) - maybe just store StringRef itself ? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117644