[PATCH] D47755: [RISCV] Insert R_RISCV_ALIGN relocation type and Nops for alignment when linker relaxation enabled

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 16 06:40:57 PDT 2018


asb added a comment.
Herald added subscribers: jocewei, PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, jrtc27.

My main concern with this patch currently is the ability to handle the relax option changing due to .option relax/norelax. So it's probably blocked on https://reviews.llvm.org/D46423.

Note that https://reviews.llvm.org/D45961 which adds MCSubtargetInfo to MCAlignFragment may also be relevant.


Repository:
  rL LLVM

https://reviews.llvm.org/D47755





More information about the llvm-commits mailing list