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

Okay, LGTM. (Sorry for the delay, I've been out.)

In D103461#2793254 <https://reviews.llvm.org/D103461#2793254>, @jansvoboda11 
wrote:

> My reason for the FIXME is that we could get rid of bunch of Windows-specific 
> logic by adjusting `CompilerInvocation` instead.

Please add that motivation to the FIXME itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103461

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

Reply via email to