This is an archive of the discontinued LLVM Phabricator instance.

[LangRef] Clarify Varargs forwarding for musttail calls.
ClosedPublic

Authored by fhahn on Jan 9 2018, 6:15 AM.

Details

Summary

This clarification was suggested by @efriedma in D41335, which uses this
behavior to inline musttail calls with varargs.

Diff Detail

Repository
rL LLVM

Event Timeline

fhahn created this revision.Jan 9 2018, 6:15 AM
efriedma added inline comments.
docs/LangRef.rst
9037 ↗(On Diff #129072)

Do we need to fix this paragraph?

9053 ↗(On Diff #129072)

It would probably be more clear to separate out the bitcasting part; it isn't really a rule, just a consequence of the way bitcasting works.

fhahn updated this revision to Diff 129263.Jan 10 2018, 5:40 AM

Thanks Eli! I've updated the initial paragraph to be explicit about what happens with varargs for tail/musttail. It is slightly more verbose, but I hope it is clearer now. I have left the varargs type rule unchanged for now.

rnk accepted this revision.Jan 17 2018, 10:14 AM

lgtm

This revision is now accepted and ready to land.Jan 17 2018, 10:14 AM
This revision was automatically updated to reflect the committed changes.