[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-16 Thread jiwang at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 Jiong Wang changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-16 Thread jiwang at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #16 from Jiong Wang --- Author: jiwang Date: Fri Jan 16 11:48:00 2015 New Revision: 219723 URL: https://gcc.gnu.org/viewcvs?rev=219723&root=gcc&view=rev Log: [AArch64] Enable CCMP support for AArch64, PR64015 resolved gcc/ 2015-01-1

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #14 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.c

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #15 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.c

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #13 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.c

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #12 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.c

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #11 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.c

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #10 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.c

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #9 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.co

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2015-01-13 Thread StaffLeavers at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #8 from StaffLeavers at arm dot com --- zhenqiang.chen no longer works for ARM. Your email will be forwarded to their line manager. Please do not reply to this email. If you need more information, please email real-postmas...@arm.co

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-26 Thread zhenqiang.chen at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #7 from Zhenqiang Chen --- Sorry for blocking your benchmark tests. I had reverted the ccmp patch. I will rework the patch based on Richard Henderson's comments: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg03100.html

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-24 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #6 from ktkachov at gcc dot gnu.org --- By the way, this ICE manifests when building perlbmk in SPEC2006

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-23 Thread zhenqiang.chen at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #5 from Zhenqiang Chen --- It seams you always win with ccmp. Please go ahead for your patch and make sure the following case work. int test (unsigned short a, unsigned char b) { return a > 0xfff2 && b > 252; } Thanks! -Zhenqiang

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-23 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #4 from Andrew Pinski --- (In reply to Andrew Pinski from comment #3) > See how with forcing is always the same size or smaller? Actually is always smaller by at least one instruction. due to the need to do one extra cset and one an

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-23 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #3 from Andrew Pinski --- (In reply to Zhenqiang Chen from comment #2) > 2) How to justify it is valueable (the overhead of ccmp is OK) when > generating ccmp? If we ignore the case for swapping. Try this one: int test (int a, int b)

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-23 Thread zhenqiang.chen at arm dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 --- Comment #2 from Zhenqiang Chen --- You force it to register? In fact, I tend to not force it to register in gen_ccmp_next, since it will introduce more overhead for ccmp, which performance maybe worse. My patch to fix the issue is at: https:

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-21 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 Andrew Pinski changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed|

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-21 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 Richard Biener changed: What|Removed |Added Priority|P3 |P1

[Bug target/64015] [5.0 Regression] AArch64 ICE due to conditional compare

2014-11-21 Thread ktkachov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64015 ktkachov at gcc dot gnu.org changed: What|Removed |Added Known to work||4.8.4, 4.9.2 Target Miles