[PATCH] D46630: [RISCV] Insert NOPs and R_RISCV_ALIGN relocation type for .align directive when linker relaxation enabled

Shiva Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 4 19:31:33 PDT 2018


shiva0217 added a comment.

Hi Alex. I find a way to emit fixup_riscv_align without introducing PseudoNOPs  as you suggest. 
The revision https://reviews.llvm.org/D47755 introducing the new target hooks to calculate the nops we need to insert and insert fixup_riscv_align for alignment directive. 
Could you help me to check the new approach is ok or not?
Thanks.


Repository:
  rL LLVM

https://reviews.llvm.org/D46630





More information about the llvm-commits mailing list