On Tue, Dec 15, 2015 at 2:33 AM, Wilco Dijkstra <wilco.dijks...@arm.com> wrote: > ping > >> -----Original Message----- >> From: Wilco Dijkstra [mailto:wilco.dijks...@arm.com] >> Sent: 13 November 2015 16:03 >> To: 'gcc-patches@gcc.gnu.org' >> Subject: [PATCH 4/4][AArch64] Cost CCMP instruction sequences to choose >> better expand order >> >> This patch adds CCMP selection based on rtx costs. This is based on Jiong's >> already approved patch https://gcc.gnu.org/ml/gcc- >> patches/2015-09/msg01434.html with some minor refactoring and the tests >> updated. >> >> OK for commit? >> >> ChangeLog: >> 2015-11-13 Jiong Wang <jiong.w...@arm.com> >> >> gcc/ >> * ccmp.c (expand_ccmp_expr_1): Cost the instruction sequences >> generated from different expand order. >>
It breaks bootstrap on Linux/x86: https://gcc.gnu.org/ml/gcc-regression/2016-01/msg00332.html -- ../../src-trunk/gcc/ccmp.c: In function ârtx_def* expand_ccmp_expr_1(gimple*, rtx_def**, rtx_def**)â: ../../src-trunk/gcc/ccmp.c:173:14: error: âretâ may be used uninitialized in this function [-Werror=maybe-uninitialized] rtx tmp2, ret, ret2; ^~~ cc1plus: all warnings being treated as errors Makefile:1085: recipe for target 'ccmp.o' failed make[6]: *** [ccmp.o] Error 1 H.J.
