sfertile added inline comments.

================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:750
+  // If we can use a plt entry as the symbol address we can assume it
+  // is local.
+  if (isa<FunctionDecl>(D) && !CGOpts.NoPLT)
----------------
I don't think this is the case. I think this would break ppc, where we need to 
restore the toc pointer after the plt stubs returns to the original call site.


https://reviews.llvm.org/D41318



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D41318: ... Rafael Avila de Espindola via Phabricator via cfe-commits
    • [PATCH] D41... Rafael Avila de Espindola via Phabricator via cfe-commits
    • [PATCH] D41... Sean Fertile via Phabricator via cfe-commits
    • [PATCH] D41... Sean Fertile via Phabricator via cfe-commits
      • Re: [PA... Rafael Avila de Espindola via cfe-commits
    • [PATCH] D41... Rafael Avila de Espindola via Phabricator via cfe-commits
    • [PATCH] D41... Sean Fertile via Phabricator via cfe-commits
    • [PATCH] D41... Rafael Avila de Espindola via Phabricator via cfe-commits

Reply via email to