[PATCH] D142278: [LoongArch] Allow %pc_lo12 relocs in JIRL's immediate operand position

Sat Jan 21 05:18:43 PST 2023

xen0n added a comment.

In D142278#4071001 <https://reviews.llvm.org/D142278#4071001>, @xry111 wrote:

> LGTM though I really dislike depending on some undocumented behavior of PCALA_LO12.

I dislike it too (a longer rant can be seen in the LLD patch btw), but it's already too late to change: gcc-13 is already at Stage 4 and glibc really had to be changed to allow linking Firefox/Chromium. It's a pity that your PR <https://github.com/loongson/LoongArch-Documentation/pull/69> was ignored, but we might be able to pull off the transition eventually after 1 or 2 years, given the current trend of new world adoption and mainstream distros' porting progress. Not all is lost ;-)

> Happe new Lunar year!

Happy new year! 春节快乐！

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142278