chrib added inline comments.
================
Comment at: lib/Driver/ToolChains/Clang.cpp:569
+ if (Triple.getEnvironment() == llvm::Triple::EABI) {
+ switch (Triple.getArch()) {
----------------
efriedma wrote:
> Specifically checking for "llvm::Triple::EABI" is suspicious... what are you
> trying to distinguish?
I'm targeting the AAPCS for bare toolsets, (we could also test EABIHF by the
way)
I'm not sure about the other ABIs (such as llvm::Triple::OpenBSD) so it is
probably conservative and stick to what I can test. Do you think this pertains
to more, for instance to AAPCS-LINUX, without breaking anything ?
https://reviews.llvm.org/D31972
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
