jansvoboda11 added a comment. In D103461#2815913 <https://reviews.llvm.org/D103461#2815913>, @dexonsmith wrote:
> 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. No problem, thanks for the review(s)! 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