[PATCH] D43545: [Driver] Make -fno-common default for Fuchsia

2018-02-23 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL325945: [Driver] Make -fno-common default for Fuchsia (authored by phosek, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D43545?vs=135193&id=

[PATCH] D43545: [Driver] Make -fno-common default for Fuchsia

2018-02-21 Thread Jake Ehrlich via Phabricator via cfe-commits
jakehehrlich accepted this revision. jakehehrlich added a comment. This revision is now accepted and ready to land. LGTM Repository: rC Clang https://reviews.llvm.org/D43545 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llv

[PATCH] D43545: [Driver] Make -fno-common default for Fuchsia

2018-02-20 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added reviewers: mcgrathr, jakehehrlich. Herald added a subscriber: cfe-commits. We never want to generate common symbols on Fuchsia. Repository: rC Clang https://reviews.llvm.org/D43545 Files: clang/lib/Driver/ToolChains/Clang.cpp clang/test/Driver/f