[PATCH] D42416: [Driver] Add support for mips32 and scudo

2018-01-25 Thread Simon Dardis via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL323412: [Driver] Add support for mips32 and scudo (authored by sdardis, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D42416?vs=131047&id=131

[PATCH] D42416: [Driver] Add support for mips32 and scudo

2018-01-24 Thread Simon Dardis via Phabricator via cfe-commits
sdardis marked an inline comment as done. sdardis added a comment. Thanks, will fix the nit on commit. Repository: rC Clang https://reviews.llvm.org/D42416 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mail

[PATCH] D42416: [Driver] Add support for mips32 and scudo

2018-01-23 Thread Kostya Kortchinsky via Phabricator via cfe-commits
cryptoad added inline comments. Comment at: lib/Driver/ToolChains/Linux.cpp:850 + const bool IsMIPS = getTriple().getArch() == llvm::Triple::mips || +getTriple().getArch() == llvm::Triple::mipsel; const bool IsMIPS64 = getTriple().getArch() == llvm::Tr

[PATCH] D42416: [Driver] Add support for mips32 and scudo

2018-01-23 Thread Kostya Kortchinsky via Phabricator via cfe-commits
cryptoad accepted this revision. cryptoad added a comment. Cool thanks! Repository: rC Clang https://reviews.llvm.org/D42416 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D42416: [Driver] Add support for mips32 and scudo

2018-01-23 Thread Simon Atanasyan via Phabricator via cfe-commits
atanasyan accepted this revision. atanasyan added a comment. This revision is now accepted and ready to land. LGTM Repository: rC Clang https://reviews.llvm.org/D42416 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/

[PATCH] D42416: [Driver] Add support for mips32 and scudo

2018-01-23 Thread Simon Dardis via Phabricator via cfe-commits
sdardis created this revision. sdardis added reviewers: cryptoad, atanasyan. Herald added a subscriber: arichardson. r317337 missed that scudo is supported on MIPS32, so permit that option for MIPS32. Repository: rC Clang https://reviews.llvm.org/D42416 Files: lib/Driver/ToolChains/Linux.c