[PATCH] D94137: [coroutine] update promise object's final layout index

Xun Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 10:26:25 PST 2021


lxfind added inline comments.


================
Comment at: llvm/lib/Transforms/Coroutines/CoroFrame.cpp:750
       FrameData.setFieldIndex(
           PromiseAlloca, B.addFieldForAlloca(PromiseAlloca, /*header*/ true));
 
----------------
PromiseAlloca is added here as a header field, which means it should have a fixed index?
Could it be there are bugs in performOptimizedStructLayout that moved it around somehow?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94137



More information about the llvm-commits mailing list