rnk accepted this revision. rnk added a comment. This revision is now accepted and ready to land.
lgtm I think the old VS Express editions used to exclude atlmfc, so there's the possibility that we'll be searching non-existent directories, but I think that's OK. Whoever wrote this code was probably basing it on what VS express did. That edition is long gone, and we're better off with this new logic. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68736/new/ https://reviews.llvm.org/D68736 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits