[PATCH] D80661: [CodeGen] Enable tail call position check for speculatable functions

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 2 13:12:50 PDT 2020


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LGTM, one nit below.



================
Comment at: llvm/test/CodeGen/PowerPC/tailcall-speculatable-callee.ll:60
+
+; Callee should not be tail called since it is not at a tail call position.
+define dso_local double @speculatable_callee_intermediate_instructions (double* nocapture %res, double %a) #0 {
----------------
Please add a FIXME explaining this call could be moved to make it a tail call.


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

https://reviews.llvm.org/D80661





More information about the llvm-commits mailing list