public inbox for gcc-bugs@sourceware.org help / color / mirror / Atom feed
* [Bug target/52137] New: bdver2 scheduler needs to be added to bdver1 insn reservations @ 2012-02-06 17:07 qneill at gcc dot gnu.org 2012-02-06 17:37 ` [Bug target/52137] " pinskia at gcc dot gnu.org ` (2 more replies) 0 siblings, 3 replies; 4+ messages in thread From: qneill at gcc dot gnu.org @ 2012-02-06 17:07 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52137 Bug #: 52137 Summary: bdver2 scheduler needs to be added to bdver1 insn reservations Classification: Unclassified Product: gcc Version: 4.7.0 Status: UNCONFIRMED Severity: blocker Priority: P3 Component: target AssignedTo: unassigned@gcc.gnu.org ReportedBy: qneill@gcc.gnu.org Created attachment 26585 --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=26585 Add bdver2 to list of cpu attributes for bdver1.md insn reservations The bdver2 scheduler lacks insn reservation conditions for cpu attributes in the bdver1.md model. The resulting generated code regresses on bdver2 hardware for SPEC and polyhedron. The attached patch recovers most of this performance regression. ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/52137] bdver2 scheduler needs to be added to bdver1 insn reservations 2012-02-06 17:07 [Bug target/52137] New: bdver2 scheduler needs to be added to bdver1 insn reservations qneill at gcc dot gnu.org @ 2012-02-06 17:37 ` pinskia at gcc dot gnu.org 2012-02-21 15:21 ` qneill at gcc dot gnu.org 2012-02-22 15:59 ` qneill at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: pinskia at gcc dot gnu.org @ 2012-02-06 17:37 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52137 Andrew Pinski <pinskia at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |missed-optimization Severity|blocker |normal ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/52137] bdver2 scheduler needs to be added to bdver1 insn reservations 2012-02-06 17:07 [Bug target/52137] New: bdver2 scheduler needs to be added to bdver1 insn reservations qneill at gcc dot gnu.org 2012-02-06 17:37 ` [Bug target/52137] " pinskia at gcc dot gnu.org @ 2012-02-21 15:21 ` qneill at gcc dot gnu.org 2012-02-22 15:59 ` qneill at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: qneill at gcc dot gnu.org @ 2012-02-21 15:21 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52137 --- Comment #1 from Quentin Neill <qneill at gcc dot gnu.org> 2012-02-21 15:15:48 UTC --- Author: qneill Date: Tue Feb 21 15:15:42 2012 New Revision: 184440 URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=184440 Log: 2012-02-21 Quentin Neill <quentin.neill@amd.com> PR target/52137 * config/i386/bdver1.md (bdver1_call, bdver1_push, bdver1_pop, bdver1_leave, bdver1_lea, bdver1_imul_DI, bdver1_imul, bdver1_imul_mem_DI, bdver1_imul_mem, bdver1_idiv, bdver1_idiv_mem, bdver1_str, bdver1_idirect, bdver1_ivector, bdver1_idirect_loadmov, bdver1_idirect_load, bdver1_ivector_load, bdver1_idirect_movstore, bdver1_idirect_both, bdver1_ivector_both, bdver1_idirect_store, bdver1_ivector_store, bdver1_fldxf, bdver1_fld, bdver1_fstxf, bdver1_fst, bdver1_fist, bdver1_fmov_bdver1, bdver1_fadd_load, bdver1_fadd, bdver1_fmul_load, bdver1_fmul, bdver1_fsgn, bdver1_fdiv_load, bdver1_fdiv, bdver1_fpspc_load, bdver1_fpspc, bdver1_fcmov_load, bdver1_fcmov, bdver1_fcomi_load, bdver1_fcomi, bdver1_fcom_load, bdver1_fcom, bdver1_fxch, bdver1_ssevector_avx128_unaligned_load, bdver1_ssevector_avx256_unaligned_load, bdver1_ssevector_sse128_unaligned_load, bdver1_ssevector_avx128_load, bdver1_ssevector_avx256_load, bdver1_ssevector_sse128_load, bdver1_ssescalar_movq_load, bdver1_ssescalar_vmovss_load, bdver1_ssescalar_sse128_load, bdver1_mmxsse_load, bdver1_sse_store_avx256, bdver1_sse_store, bdver1_mmxsse_store_short, bdver1_ssevector_avx256, bdver1_movss_movsd, bdver1_mmxssemov, bdver1_sselog_load_256, bdver1_sselog_256, bdver1_sselog_load, bdver1_sselog, bdver1_ssecmp_load, bdver1_ssecmp, bdver1_ssecomi_load, bdver1_ssecomi, bdver1_vcvtX2Y_avx256_load, bdver1_vcvtX2Y_avx256, bdver1_ssecvt_cvtss2sd_load, bdver1_ssecvt_cvtss2sd, bdver1_sseicvt_cvtsi2sd_load, bdver1_sseicvt_cvtsi2sd, bdver1_ssecvt_cvtpd2ps_load, bdver1_ssecvt_cvtpd2ps, bdver1_ssecvt_cvtdq2ps_load, bdver1_ssecvt_cvtdq2ps, bdver1_ssecvt_cvtdq2pd_load, bdver1_ssecvt_cvtdq2pd, bdver1_ssecvt_cvtps2pd_load, bdver1_ssecvt_cvtps2pd, bdver1_ssecvt_cvtsX2si_load, bdver1_ssecvt_cvtsX2si, bdver1_ssecvt_cvtpd2pi_load, bdver1_ssecvt_cvtpd2pi, bdver1_ssecvt_cvtpd2dq_load, bdver1_ssecvt_cvtpd2dq, bdver1_ssecvt_cvtps2pi_load, bdver1_ssecvt_cvtps2pi, bdver1_ssemuladd_load_256, bdver1_ssemuladd_256, bdver1_ssemuladd_load, bdver1_ssemuladd, bdver1_sseimul_load, bdver1_sseimul, bdver1_sseiadd_load, bdver1_sseiadd, bdver1_ssediv_double_load_256, bdver1_ssediv_double_256, bdver1_ssediv_single_load_256, bdver1_ssediv_single_256, bdver1_ssediv_double_load, bdver1_ssediv_double, bdver1_ssediv_single_load, bdver1_ssediv_single, bdver1_sseins): Add "bdver2" attribute. Modified: trunk/gcc/ChangeLog trunk/gcc/config/i386/bdver1.md ^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/52137] bdver2 scheduler needs to be added to bdver1 insn reservations 2012-02-06 17:07 [Bug target/52137] New: bdver2 scheduler needs to be added to bdver1 insn reservations qneill at gcc dot gnu.org 2012-02-06 17:37 ` [Bug target/52137] " pinskia at gcc dot gnu.org 2012-02-21 15:21 ` qneill at gcc dot gnu.org @ 2012-02-22 15:59 ` qneill at gcc dot gnu.org 2 siblings, 0 replies; 4+ messages in thread From: qneill at gcc dot gnu.org @ 2012-02-22 15:59 UTC (permalink / raw) To: gcc-bugs http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52137 Quentin Neill <qneill at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution| |FIXED --- Comment #2 from Quentin Neill <qneill at gcc dot gnu.org> 2012-02-22 15:05:54 UTC --- This patch restores the expected performance for -march=bdver2 ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-02-22 15:10 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-02-06 17:07 [Bug target/52137] New: bdver2 scheduler needs to be added to bdver1 insn reservations qneill at gcc dot gnu.org 2012-02-06 17:37 ` [Bug target/52137] " pinskia at gcc dot gnu.org 2012-02-21 15:21 ` qneill at gcc dot gnu.org 2012-02-22 15:59 ` qneill at gcc dot gnu.org
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).