[PATCH] D126715: [coro async] Add code to support dynamic aligment of over-aligned types in async frames

[John McCall via Phabricator via llvm-commits]

rjmccall added inline comments.


================
Comment at: llvm/lib/Transforms/Coroutines/CoroFrame.cpp:1809
     for (const auto &Alias : A.Aliases) {
-      auto *FramePtr = GetFramePointer(Alloca);
+      auto *FramePtr = GetFramePointer(Alloca, false /*NeedsDynamicAlignment*/);
       auto *FramePtrRaw =
----------------
These three sites all need to be `A.NeedsDynamicAlignment` in case the original alloca is over-aligned, right?  Should we just make `GetFramePointer` take a `const Field &` so that this falls out?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126715