[PATCH] D58579: [Sema] SequenceChecker: C++17 sequencing rule for call expressions.
Bruno Ricci via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 17 09:45:45 PST 2019
riccibruno added a comment.
In D58579#1788164 <https://reviews.llvm.org/D58579#1788164>, @riccibruno wrote:
> In D58579#1788125 <https://reviews.llvm.org/D58579#1788125>, @xbolva00 wrote:
>
> > Maybe @aaron.ballman / @rsmith could take a look? This patch blocks the whole patch stack.
>
>
> I rebased it on top of the others already accepted patches. I just need to get back +w access to the GitHub repo to commit them, but this patch is not blocking them anymore.
>
> Note that I am not entirely sure that the implementation in `getMemoryLocation` is the right way to do this; I think that this patch should be considered a work-in-progress.
Duh, I meant to post this comment in D57660 <https://reviews.llvm.org/D57660> instead. Still this patch is not blocking the other already accepted patches.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58579/new/
https://reviews.llvm.org/D58579
More information about the cfe-commits
mailing list