[PATCH] D74385: [ARCMT][NFC] Reduce #include dependencies

2020-02-12 Thread serge via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG53fba65d2204: [ARCMT][NFC] Reduce #include dependencies (authored by nicolas17, committed by serge-sans-paille). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D74385: [ARCMT][NFC] Reduce #include dependencies

2020-02-12 Thread Nicolás Alvarez via Phabricator via cfe-commits
nicolas17 added a comment. In D74385#1872298 , @serge-sans-paille wrote: > Sure, can you just confirm that you're this guy: https://github.com/nicolas17 > (I need that for proper email attribution) Yes that's me :) Repository: rG LLVM Github

[PATCH] D74385: [ARCMT][NFC] Reduce #include dependencies

2020-02-12 Thread serge via Phabricator via cfe-commits
serge-sans-paille added a comment. In D74385#1869876 , @nicolas17 wrote: > I don't have commit access, can someone push this for me? Sure, can you just confirm that you're this guy: https://github.com/nicolas17 (I need that for proper email

[PATCH] D74385: [ARCMT][NFC] Reduce #include dependencies

2020-02-11 Thread Nicolás Alvarez via Phabricator via cfe-commits
nicolas17 added a comment. I don't have commit access, can someone push this for me? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74385/new/ https://reviews.llvm.org/D74385 ___ cfe-commits mailing

[PATCH] D74385: [ARCMT][NFC] Reduce #include dependencies

2020-02-11 Thread serge via Phabricator via cfe-commits
serge-sans-paille added a comment. LGTM, as long as it builds fine. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74385/new/ https://reviews.llvm.org/D74385 ___ cfe-commits mailing list

[PATCH] D74385: [ARCMT][NFC] Reduce #include dependencies

2020-02-10 Thread Nicolás Alvarez via Phabricator via cfe-commits
nicolas17 created this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Replace some #includes in ARCMigrate source files with more specific includes and forward declarations. This reduces the number of files that need to be rebuilt when a header changes (and