ychen added a comment.

In D128750#3616471 <https://reviews.llvm.org/D128750#3616471>, @ychen wrote:

> In D128750#3616449 <https://reviews.llvm.org/D128750#3616449>, @erichkeane 
> wrote:
>
>> Is there any chance you can validate this against 
>> https://reviews.llvm.org/D126907 as well? We touch similar code, and I'm 
>> intending to get that committed in the near future.  Otherwise, after a 
>> quick look, I don't see anything particualrly bad, other than a lack of 
>> release notes and update of www-status.
>
> Sure. I will test it.

`check-all` passes with this patch + D126907 <https://reviews.llvm.org/D126907>.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128750

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

Reply via email to