lebedev.ri added a comment.

In D60760#1487387 <https://reviews.llvm.org/D60760#1487387>, @sberg wrote:

> In D60760#1487342 <https://reviews.llvm.org/D60760#1487342>, @lebedev.ri 
> wrote:
>
> > Did this get reviewed?
>
>
> I didn't get any responses at all, so decided to push it anyway.


That is not really how reviews work in LLVM.
Either it's a NFC / eligible for post-commit review, and can just be committed, 
or not.
E.g., is this missing test coverage?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60760



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

Reply via email to