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

Reply via email to