In working on the vector extracts, I noticed that the function altivec_resolve_overloaded_builtin has two strings that span more than one line. This confuses the diff -p command, as it thinks modified code after the long line is in another function.
This patch fixes the formatting. I did a bootstrap and make check, and there were no regressions. Is it ok to change into the trunk? 2016-07-22 Michael Meissner <meiss...@linux.vnet.ibm.com> * config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin): Reformat two multi-line strings. Index: gcc/config/rs6000/rs6000-c.c =================================================================== --- gcc/config/rs6000/rs6000-c.c (revision 238490) +++ gcc/config/rs6000/rs6000-c.c (working copy) @@ -4731,11 +4731,13 @@ altivec_resolve_overloaded_builtin (loca /* vec_lvsl and vec_lvsr are deprecated for use with LE element order. */ if (fcode == ALTIVEC_BUILTIN_VEC_LVSL && !VECTOR_ELT_ORDER_BIG) - warning (OPT_Wdeprecated, "vec_lvsl is deprecated for little endian; use \ -assignment for unaligned loads and stores"); + warning (OPT_Wdeprecated, + "vec_lvsl is deprecated for little endian; use " + "assignment for unaligned loads and stores"); else if (fcode == ALTIVEC_BUILTIN_VEC_LVSR && !VECTOR_ELT_ORDER_BIG) - warning (OPT_Wdeprecated, "vec_lvsr is deprecated for little endian; use \ -assignment for unaligned loads and stores"); + warning (OPT_Wdeprecated, + "vec_lvsr is deprecated for little endian; use " + "assignment for unaligned loads and stores"); if (fcode == ALTIVEC_BUILTIN_VEC_MUL) { -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meiss...@linux.vnet.ibm.com, phone: +1 (978) 899-4797