[PATCH] D63030: [WebAssembly] Modernize include path handling

2019-06-13 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL363241: [WebAssembly] Modernize include path handling (authored by sbc, committed by ). Herald added subscribers: llvm-commits, ormris. Herald added a project: LLVM. Changed prior to commit: https://rev

[PATCH] D63030: [WebAssembly] Modernize include path handling

2019-06-08 Thread Dan Gohman via Phabricator via cfe-commits
sunfish accepted this revision. sunfish added a comment. This revision is now accepted and ready to land. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63030/new/ https://reviews.llvm.org/D63030

[PATCH] D63030: [WebAssembly] Modernize include path handling

2019-06-07 Thread Sam Clegg via Phabricator via cfe-commits
sbc100 updated this revision to Diff 203634. sbc100 added a comment. - more OS Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D63030/new/ https://reviews.llvm.org/D63030 Files: clang/lib/Driver/ToolChains/WebAssembly.cpp clang/lib/Frontend/InitH

[PATCH] D63030: [WebAssembly] Modernize include path handling

2019-06-07 Thread Sam Clegg via Phabricator via cfe-commits
sbc100 created this revision. Herald added subscribers: cfe-commits, sunfish, aheejin, jgravelle-google, dschuff. Herald added a project: clang. Move include path construction from InitHeaderSearch::AddDefaultIncludePaths in the Driver which appears to be the more modern/correct way of doing thin