[PATCH] D67855: [X86] Add new calling convention that guarantees tail call optimization
Dwight Guth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 24 09:25:50 PDT 2019
dwightguth added a comment.
I added tests for musttail and mixed calling conventions. But I'm a little unsure what the correct behavior should be here for disable-tail-calls. When you do a musttail call, disable-tail-calls is ignored. When you do a fastcc call with -tailcallopt, disable-tail-calls is not ignored. This seems a little inconsistent to me, but is beyond the scope of this revision. Which of these two behaviors should be implemented by tailcc, though? Currently it does the same thing as fastcc with -tailcallopt. I can add a test for that behavior, but if we want the other behavior, I need to know before I write that test because it affects the test's output.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67855/new/
https://reviews.llvm.org/D67855
More information about the llvm-commits
mailing list