aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

In D94624#2501894 <https://reviews.llvm.org/D94624#2501894>, @tomrittervg wrote:

> I started trying to work on a patch, but I'm still unpacking from a move and 
> don't have all my machines set up - trying to enable and build tests filled 
> up my hard drive (even after I removed the easy-to-remove stuff), so I don't 
> think I'm going to be able to create a test for this in the short-term.

Okay, I think this LGTM even without the test coverage. Thank you for the 
patch, do you need someone to commit it on your behalf?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94624

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

Reply via email to