[PATCH] D90301: [yaml2obj][test] - Merge dynsymtab-shlink.yaml to dynsym-section.yaml

[James Henderson via Phabricator via llvm-commits]

jhenderson added inline comments.


================
Comment at: llvm/test/tools/yaml2obj/ELF/dynsym-section.yaml:57
+
+## Check that by default thr .dynsym section will be linked to the .dynstr section,
+## when the latter one exists.
----------------



================
Comment at: llvm/test/tools/yaml2obj/ELF/dynsym-section.yaml:60
+
+# RUN: yaml2obj %s --docnum=2 -DLINK="" -o %t3
+# RUN: llvm-readelf --section-headers %t3 | FileCheck %s --check-prefix=LINK-DEFAULT
----------------
If you used Link: `[[LINK:<none>]]` (or whatever the syntax is), that'll allow you to avoid the `-DLINK=""` bit here, right?


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

https://reviews.llvm.org/D90301