[PATCH] D77330: Consider increasing the default ARM32 page size to 64k.

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 12:41:12 PDT 2020


psmith added a comment.

I've uploaded a fixed arm-branch-rangethunks.s F11737458: D77330.diff <https://reviews.llvm.org/F11737458>  I took out the modification to raise the .balign to 0x20000 as I don't think it is necessary. I updated the comment and used --no-show-raw-insn in objdump -d as this makes it easier to read. Apart from that the only difference is that the .text section starts at 0x30000 instead of 0x20000 so I needed to add 0x10000 to the far-arm-abs.s and far-long-arm.s values. Can you check the attachment works and add it to the patch?

I'll try and get to the other 2 failing tests later this week or the weekend.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77330





More information about the llvm-commits mailing list