Anastasia added a comment. LGTM! Thanks
I agree we should have the same behavior for both options. Technically `-finclude-default-header` could be added in both steps but it is only effective in parsing the sources and does nothing otherwise. But it seems cleaner to align it with the header include option. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D104800/new/ https://reviews.llvm.org/D104800 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits