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
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
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
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
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/
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