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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to