[PATCH] D51493: [llvm-strip] Allow copying relocation sections without symbol tables.

[Alexander Shaposhnikov via Phabricator via llvm-commits]

alexshap added inline comments.


================
Comment at: tools/llvm-objcopy/Object.cpp:396
 void RelocSectionWithSymtabBase<SymTabType>::finalize() {
-  this->Link = Symbols->Index;
+  if (Symbols != nullptr)
+    this->Link = Symbols->Index;
----------------
 to double check - Link is not getting set at all (if Symbols == nullptr),  just wondering what the value will be in this case ? 0 or garbage ? probably i'd do what binutils objcopy does.  Asking just in case.


Repository:
  rL LLVM

https://reviews.llvm.org/D51493