[PATCH] D77200: [LLD][ELF][ARM] use reloc directive for Thumb assembler tests

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 1 08:27:58 PDT 2020


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

LGTM.

> use reloc directive for Thumb assembler tests

Maybe `reloc` -> `.reloc`

> I noticed that .reloc wasn't behaving well when an Alignment or Padding fragment had been inserted. I've replaced the few inline .balign directives with nops in this case. Will need to look into MC and raise a PR if there is a problem.

I can't find where nops are used...


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

https://reviews.llvm.org/D77200





More information about the llvm-commits mailing list