[PATCH] D81126: [PowerPC] Fix for PC Relative call protocol

[Victor Huang via Phabricator via llvm-commits]

NeHuang added a comment.

nit: In SUMMARY  "add the cnop" to "add the nop"



================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:4696
+    if (!STICaller->isUsingPCRelativeCalls() &&
+        STICallee->isUsingPCRelativeCalls())
+      return false;
----------------
NeHuang wrote:
> The fix makes sense to me.  
> 
> For the other two scenarios:
> 1. Caller using PCRelative addressing, callee using TOC.
> 2. Caller using PCRelative addressing, callee using PCRelative addressing.
> 
> In both cases, is it possible TOC could be clobbered in the caller (same as the test case below when it call external function externalFunc)?  Can we say caller and callee are not sharing the same TOC base?
> If so, caller and callee are not share the same TOC but we do not need nop to restore the TOC in the caller. 
> 
> In this sense, shall we rename this function to `callDoesNotRequireTOCRestore` since the previous assumption no longer hold for PCRelative addressing?
please ignore this one and found the comment in line 4664


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

https://reviews.llvm.org/D81126