[PATCH] D60042: [llvm-objcopy] Add --prefix-alloc-sections

Seiya Nuta via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 02:02:39 PDT 2019


seiya marked an inline comment as done.
seiya added inline comments.


================
Comment at: llvm/tools/llvm-objcopy/ELF/ELFObjcopy.cpp:637
+                const SectionRename &SR = Iter->second;
+                Sec.Name =
+                    (prefix + Config.AllocSectionsPrefix + SR.NewName).str();
----------------
MaskRay wrote:
> How about `Sec.Name = (prefix + Config.AllocSectionsPrefix + Iter->second.NewName).str();`
Looks better. Thanks.


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

https://reviews.llvm.org/D60042





More information about the llvm-commits mailing list