[PATCH] D66982: [Modules][Objective-C] Use complete decl from module when diagnosing missing import

2019-09-16 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL372039: [Modules][Objective-C] Use complete decl from module when diagnosing missing… (authored by bruno, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed pri

[PATCH] D66982: [Modules][Objective-C] Use complete decl from module when diagnosing missing import

2019-09-13 Thread Alex Lorenz via Phabricator via cfe-commits
arphaman accepted this revision. arphaman 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/D66982/new/ https://reviews.llvm.org/D66982 ___

[PATCH] D66982: [Modules][Objective-C] Use complete decl from module when diagnosing missing import

2019-09-13 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
bruno added a comment. Ping! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66982/new/ https://reviews.llvm.org/D66982 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[PATCH] D66982: [Modules][Objective-C] Use complete decl from module when diagnosing missing import

2019-08-29 Thread Bruno Cardoso Lopes via Phabricator via cfe-commits
bruno created this revision. bruno added reviewers: rsmith, arphaman. Herald added subscribers: cfe-commits, ributzka, dexonsmith, jkorous. Herald added a project: clang. Otherwise the definition (first found) for ObjCInterfaceDecl's might precede the module one, which will eventually lead to cras