[PATCH] D48663: [Power9] Code Cleanup - Remove needsAggressiveScheduling()

Stefan Pintilie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 28 12:33:14 PDT 2018


stefanp updated this revision to Diff 153369.
stefanp added a comment.

Fixed comments according to previous review.

Also, the three tests that I wanted to delete I've actually kept. I added asm to each of those three tests to force the code to spill the condition register. If the condition register is spilled the tests can once again check to see if the spill is correct.


https://reviews.llvm.org/D48663

Files:
  lib/Target/PowerPC/PPCSubtarget.cpp
  test/CodeGen/PowerPC/2008-10-28-f128-i32.ll
  test/CodeGen/PowerPC/2011-12-05-NoSpillDupCR.ll
  test/CodeGen/PowerPC/coalesce-ext.ll
  test/CodeGen/PowerPC/fp128-bitcast-after-operation.ll
  test/CodeGen/PowerPC/i1-to-double.ll
  test/CodeGen/PowerPC/lsa.ll
  test/CodeGen/PowerPC/ppc32-vacopy.ll
  test/CodeGen/PowerPC/ppc64-gep-opt.ll
  test/CodeGen/PowerPC/save-cr-ppc32svr4.ll
  test/CodeGen/PowerPC/save-crbp-ppc32svr4.ll
  test/CodeGen/PowerPC/tls.ll
  test/CodeGen/PowerPC/unal-altivec.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48663.153369.patch
Type: text/x-patch
Size: 25643 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180628/bd9eb19f/attachment.bin>


More information about the llvm-commits mailing list