[PATCH] D81148: [MC] Fix DWARF forms for 64-bit DWARFv3 files [6/7]

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/test/MC/ELF/gen-dwarf64.s:18
 # REL5-NEXT:     R_X86_64_64 .debug_rnglists 0x14
 # REL:         Section ({{[0-9]+}}) .rela.debug_aranges {
 # REL-NEXT:      R_X86_64_64 .debug_info 0x0
----------------
ikudrin wrote:
> MaskRay wrote:
> > This `REL` can be `REL-NEXT` to prevent the accidental omission of `check-prefixes=REL,REL3`
> There are other relocations for the `.debug_info` section which I did not want to check because they are irrelevant to the changes of these patches. So, I probably would not add the `-NEXT` suffix, if you do not mind.
At least a "REL-NOT: }" to check that you haven't accidentally jumped into another section's relocations might be useful? But really that'd have to go between every one of these check lines.

Perhaps llvm-readobj has flags that specify which sections to dump - and if you only dump the relocations for the sections you're interested in, then you could use an --implicit-check-not: Section, for instance - to ensure you didn't jump over any section headers.


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

https://reviews.llvm.org/D81148