[PATCH] D48685: [PCH+Modules] Load -fmodule-map-file content before including PCHs

2018-07-19 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL337447: [PCH+Modules] Load -fmodule-map-file content before including PCHs (authored by bruno, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit:

[PATCH] D48685: [PCH+Modules] Load -fmodule-map-file content before including PCHs

2018-07-09 Thread Ben Langmuir via Phabricator via cfe-commits
benlangmuir accepted this revision. benlangmuir added a comment. This revision is now accepted and ready to land. LGTM https://reviews.llvm.org/D48685 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D48685: [PCH+Modules] Load -fmodule-map-file content before including PCHs

2018-07-06 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
bruno added a comment. Herald added a subscriber: dexonsmith. Ping! https://reviews.llvm.org/D48685 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D48685: [PCH+Modules] Load -fmodule-map-file content before including PCHs

2018-06-27 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
bruno created this revision. bruno added reviewers: rsmith, benlangmuir. Consider: 1. Generate PCH with -fmodules and -fmodule-map-file 2. Use PCH with -fmodules and the same -fmodule-map-file If we don't load -fmodule-map-file content before including PCHs, the modules that are dependencies