[PATCH] D6260: Add -mlong-double-64 flag

2018-08-27 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. What about `-mlong-double-128`? It seems sensible to me to add support for that at the same time. It looks like this also needs to affect name mangling. As far as I can tell from some quick testing, on ppc64-linux-gnu GCC uses `e` for 64-bit `long double` and `g` for

[PATCH] D6260: Add -mlong-double-64 flag

2018-06-21 Thread Hal Finkel via Phabricator via cfe-commits
hfinkel added inline comments. Comment at: lib/Basic/TargetInfo.cpp:277 +LongDoubleWidth = 64; +LongDoubleAlign = 32; +LongDoubleFormat = ::APFloat::IEEEdouble; This seems wrong for targets in general. Maybe this should be, instead of 32 always, 64

[PATCH] D6260: Add -mlong-double-64 flag

2018-06-21 Thread Chandler Carruth via Phabricator via cfe-commits
chandlerc edited subscribers, added: brooksmoses, chandlerc; removed: rnk. chandlerc edited reviewers, added: echristo, timshen, dlj; removed: kcc. chandlerc added a comment. Ok folks, I know this is a blast from the past, but can we actually review this? I'm willing to commandeer it, rebase it,