[PATCH] D66576: [Regalloc][WIP] Increase CSR cost in RegAllocGreedy to favour splitting/spill over CSR first use

Qing Shan Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 5 19:53:33 PDT 2019


steven.zhang added a comment.

.AMDGPU also override the getCSRFirstUseCost() but your patch didn't catch that. And would you please post some improve number for powerpc of this patch ?



================
Comment at: llvm/test/CodeGen/PowerPC/tail-dup-break-cfg.ll:1
-; RUN: llc -O2 -o - %s | FileCheck %s
 target datalayout = "e-m:e-i64:64-n32:64"
----------------
I suggest that, you should commit a NFC patch to update the CHCK first.


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

https://reviews.llvm.org/D66576





More information about the llvm-commits mailing list