[PATCH] D71390: [PowerPC] Modify the hasSideEffects of MTLR and MFLR from 1 to 0

qshanz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 15 18:43:19 PST 2019


steven.zhang added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/ppcf128-constrained-fp-intrinsics.ll:1389
+; PC64LE-NEXT:    ld 29, -48(1) # 8-byte Folded Reload
+; PC64LE-NEXT:    mtlr 0
 ; PC64LE-NEXT:    blr
----------------
jsji wrote:
> This is way to far from `ld 0,16(1)` now. 
But those load in between didn't have any dependency. So, it is likely an heuristic between the resource and latency. 


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

https://reviews.llvm.org/D71390





More information about the llvm-commits mailing list