This is an archive of the discontinued LLVM Phabricator instance.

[Coroutines] Remove alignment check in shouldBeMustTail
ClosedPublic

Authored by junparser on Apr 2 2020, 8:53 PM.

Details

Summary

Since align is ABI-affecting only in combination with byval, there is no need to check alignment attribute anymore in shouldBeMustTail.

TestPlan: check-llvm

Diff Detail

Event Timeline

junparser created this revision.Apr 2 2020, 8:53 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 2 2020, 8:54 PM
junparser edited the summary of this revision. (Show Details)Apr 2 2020, 8:54 PM
rjmccall accepted this revision.Apr 3 2020, 9:06 AM

LGTM.

This revision is now accepted and ready to land.Apr 3 2020, 9:06 AM
This revision was automatically updated to reflect the committed changes.