[PATCH] D81653: [ELF] Fixing an issue in Elf_Note_Impl::getDescAsStringRef

Scott Linder via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 11 14:22:29 PDT 2020


scott.linder accepted this revision.
scott.linder added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/include/llvm/Object/ELFTypes.h:620
   StringRef getDescAsStringRef() const {
-    auto &Desc = getDesc();
+    auto Desc = getDesc();
     return StringRef(reinterpret_cast<const char *>(Desc.data()), Desc.size());
----------------
Could you just use the full type here?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81653





More information about the llvm-commits mailing list