[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

Yuanfang Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 5 12:28:13 PST 2021


ychen added a comment.

In D97915#2607338 <https://reviews.llvm.org/D97915#2607338>, @rjmccall wrote:

> Let's try to avoid adding a new builtin for what we acknowledge is a workaround.  Builtins become part of the language supported by the compiler, so we shouldn't add them casually.

If we're going to use the aligned new in the future, do we still need this builtin, or something else is preferred?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97915



More information about the llvm-commits mailing list