[PATCH] D38128: Handle COPYs of physregs better (regalloc hints)

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 15 06:32:59 PST 2017


jonpa added a comment.

@Quentin: I think the patch is starting to be somewhat convincing, as it is now also confirmed and approved on PowerPC, as well as has gotten approved test changes for AMDGPU and CodeGen/ARM. Given that it's now been two months of review, and that I updated all the tests already a month ago, I wonder what you would say about activating this for now under a target preference hook? I think that SystemZ, PowerPC and AMDGPU could benefit from this now, while other targets could do so after reviewing test changes. Would this work?

Status table corrected:

  TODO:
  CodeGen   : BPF Hexagon Mips SPARC Thumb Thumb2 X86 XCore
  DebugInfo : COFF X86
  
  PARTIALLY DONE :
   CodeGen :
    - AArch64: All done but swifterror.ll (small regression and needs approval according to Martin Storsjo)
  
  APPROVED:
   CodeGen :
    - CodeGen/ARM (Eli Friedman)
    - AMDGPU (Tom Stellard)
    - PowerPC (Nemanja Ivanovic)
  
  (Regression? : test/DebugInfo/X86/live-debug-variables.ll)


https://reviews.llvm.org/D38128





More information about the llvm-commits mailing list