[PATCH] D91053: [PowerPC] Lump the constants to save one addis for each constant access

Qing Shan Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 21 02:32:47 PST 2020


steven.zhang added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/constant-pool.ll:19
+; CHECK-P9-NEXT:    ld r3, .LC0 at toc@l(r3)
+; CHECK-P9-NEXT:    lfs f1, 0(r3)
 ; CHECK-P9-NEXT:    blr
----------------
qiucf wrote:
> Regression?
Yes, we will produce an extra load if there is single constant. But the linker will optimize away this extra load.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91053



More information about the llvm-commits mailing list