[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

2020-10-21 Thread John Brawn via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG0c66606230df: [Driver] Incorporate -mfloat-abi in the computed triple on ARM (authored by john.brawn). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89573/ne

[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

2020-10-20 Thread Eli Friedman via Phabricator via cfe-commits
efriedma accepted this revision. efriedma added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89573/new/ https://reviews.llvm.org/D89573 ___ cfe-commits mailing list cfe-commits

[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

2020-10-20 Thread John Brawn via Phabricator via cfe-commits
john.brawn updated this revision to Diff 299420. john.brawn edited the summary of this revision. john.brawn added a comment. Adjusted to give an error when the specified float abi is incompatible with the default. Was a bit trickier than I expected as it required extracting out the logic to dete

[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

2020-10-16 Thread Eli Friedman via Phabricator via cfe-commits
efriedma added inline comments. Comment at: clang/lib/Driver/ToolChain.cpp:807 +default: + break; +} Do we want to error in the "default" case? Not that anyone is likely to use -mfloat-abi on those targets, but I'd prefer not to silently miscompile

[PATCH] D89573: [Driver] Incorporate -mfloat-abi in the computed triple on ARM

2020-10-16 Thread John Brawn via Phabricator via cfe-commits
john.brawn created this revision. john.brawn added reviewers: efriedma, vhscampos, chill. Herald added subscribers: dexonsmith, steven_wu, hiraditya, kristof.beyls. Herald added a project: clang. john.brawn requested review of this revision. LLVM assumes that when it creates a call to a C library