[PATCH] D77844: [ObjectYAML][MachO] Add support for relocations

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 14 12:58:23 PDT 2020


MaskRay added inline comments.


================
Comment at: llvm/test/ObjectYAML/MachO/relocations_arm64.yaml:93
+# CHECK-NEXT:         relocations:
+# CHECK-NEXT:           - address:         0x0000002C
+# CHECK-NEXT:             symbolnum:       2
----------------
Among these relocation fields, does any one have a default value and can be omitted? For example, if `scattered` is almost always false (I don't know if that is true), it can be omitted for simpler output.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77844





More information about the llvm-commits mailing list