Szelethus added a comment.

In D71524#1889566 <https://reviews.llvm.org/D71524#1889566>, @boga95 wrote:

> @steakhal's revision is on the top of this. Changing the order will only 
> cause unnecessary work on both sides.


He recently rebased on top of master. I'm no fan of creating unnecessary work 
for either of you, and it might just be the case that my worry is greater then 
necessary. But for now, landing more patches with this infrastructure and 
having to painstakingly redo them all sounds like more work.

It'd be best to wait for @NoQ's feedback, after all, he did approve earlier 
patches. And again, sorry for not being available earlier.


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

https://reviews.llvm.org/D71524



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

Reply via email to