[PATCH] D102147: [Clang][Coroutines] Implement P2014R0 Option 1 behind -fcoroutines-aligned-alloc

Yuanfang Chen via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 10 08:28:52 PDT 2021


ychen added a comment.

In D102147#2747611 <https://reviews.llvm.org/D102147#2747611>, @ChuanqiXu wrote:

> Since D97915 <https://reviews.llvm.org/D97915> would fix the problem that the variables in the frame may not be aligned, I think this option `fcoroutines-aligned-alloc` won't affect normal programmers other than language lawyers. Do you think so?

I think that's right that if all a user want is alignment, then `fcoroutines-aligned-alloc` may not be necessay. However if they define customed aligned allocators/dealllocators, this is needed to make it work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102147



More information about the cfe-commits mailing list