[PATCH] D46232: [SystemZ, IPRA] determineCalleeSaves must always add return register and DP.

Vivek Pandya via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 30 11:08:58 PDT 2018


vivekvpandya added a comment.

In https://reviews.llvm.org/D46232#1083113, @uweigand wrote:

> But we do not want to force use of a frame pointer, in general this just reduces performance for no reason on Z.   Some functions require a frame pointer (those that do dynamic allocas), but most do not.


Then I think code should be modified in following manner:

  if (HasFP || MRI.isPhysRegModified(SystemZ::R11D) )
    SavedRegs.set(SystemZ::R11D);


https://reviews.llvm.org/D46232





More information about the llvm-commits mailing list