[PATCH] D32347: Add support for openSUSE ARM Triples

2017-04-21 Thread İsmail Dönmez via Phabricator via cfe-commits
ismail created this revision. Herald added a subscriber: aemerson. Add support for armv6hl-suse-linux-gnueabi and armv7hl-suse-linux-gnueabi triples used by openSUSE/SUSE. https://reviews.llvm.org/D32347 Files: lib/Driver/ToolChains/Gnu.cpp test/Driver/Inputs/opensuse_tumbleweed_armv6hl_tr

[PATCH] D32347: Add support for openSUSE ARM Triples

2017-04-21 Thread İsmail Dönmez via Phabricator via cfe-commits
ismail added a comment. In https://reviews.llvm.org/D32347#733474, @rengolin wrote: > Nothing new here, pretty much standard. No "gnueabihf"? Yes openSUSE prefers the "hl" prefix to mean HardFloat. https://reviews.llvm.org/D32347 ___ cfe-commits

[PATCH] D32347: Add support for openSUSE ARM Triples

2017-04-21 Thread İsmail Dönmez via Phabricator via cfe-commits
ismail added a comment. In https://reviews.llvm.org/D32347#733476, @rengolin wrote: > In https://reviews.llvm.org/D32347#733475, @ismail wrote: > > > Yes openSUSE prefers the "hl" prefix to mean HardFloat. > > > I'm not sure that'll do what you expect it, though. Having the "hf" at the > end of

[PATCH] D32427: Fix float abi for SUSE ARM triples

2017-04-24 Thread İsmail Dönmez via Phabricator via cfe-commits
ismail created this revision. Herald added a subscriber: aemerson. SUSE's ARM triples always ends with -gnueabi but all the targets are hard-float https://reviews.llvm.org/D32427 Files: lib/Driver/ToolChains/Arch/ARM.cpp test/Driver/linux-as.c Index: test/Driver/linux-as.c ===

[PATCH] D32427: Fix float abi for SUSE ARM triples

2017-04-24 Thread İsmail Dönmez via Phabricator via cfe-commits
ismail updated this revision to Diff 96382. ismail added a comment. Fix typo https://reviews.llvm.org/D32427 Files: lib/Driver/ToolChains/Arch/ARM.cpp test/Driver/linux-as.c Index: test/Driver/linux-as.c === --- test/Driver/l

[PATCH] D32427: Fix float abi for SUSE ARM triples

2017-04-24 Thread İsmail Dönmez via Phabricator via cfe-commits
ismail added a comment. I am open to suggestions on how to do this correctly. The problem as stated is all SUSE ARM triples end with -gnueabi but they are all hard-float. Environment handling in LLVM seems to only check for the last part of the triple and not the whole string. https://reviews

[PATCH] D32427: Fix float abi for SUSE ARM triples

2017-04-25 Thread İsmail Dönmez via Phabricator via cfe-commits
ismail abandoned this revision. ismail added a comment. compnerd has a better fix. https://reviews.llvm.org/D32427 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits