[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