[PATCH] D102477: X86: support Swift Async context

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 14 10:24:40 PDT 2021


jroelofs accepted this revision.
jroelofs added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/lib/Target/X86/X86MachineFunctionInfo.h:116
+
+  Optional<int> SwiftAsyncContextFrameIdx;
+
----------------
I found the naming of these two a bit confusing... bit of a semantic uncanny valley, since they're almost but not quite referring to the same thing. I don't have a better suggestion though :(


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

https://reviews.llvm.org/D102477



More information about the llvm-commits mailing list