[PATCH] D105288: [coro async] Cap the alignment of spilled values (vs. allocas) at the max frame alignment

Arnold Schwaighofer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 1 07:55:59 PDT 2021


aschwaighofer created this revision.
aschwaighofer added reviewers: rjmccall, nate_chandler.
Herald added subscribers: lxfind, hiraditya.
aschwaighofer requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Before this patch we would normally use the ABI alignment which can be
to high for the context alginment.

For spilled values we don't need ABI alignment, since the frame entry's
address is not escaped.

rdar://79664965


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D105288

Files:
  llvm/lib/Transforms/Coroutines/CoroFrame.cpp
  llvm/test/Transforms/Coroutines/coro-async.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D105288.355883.patch
Type: text/x-patch
Size: 4357 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210701/da2e3a8d/attachment.bin>


More information about the llvm-commits mailing list