[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