This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[PATCH] Fix INSN_TICK heuristic for SCHED_PRESSURE

From: Maxim Kuvyrkov <maxim dot kuvyrkov at linaro dot org>
To: GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: Vladimir Makarov <vmakarov at redhat dot com>, Jeff Law <law at redhat dot com>
Date: Mon, 14 Jul 2014 16:12:04 +1200
Subject: [PATCH] Fix INSN_TICK heuristic for SCHED_PRESSURE
Authentication-results: sourceware.org; auth=none

Hi,

This patch fixes 2 of the [several] problems in rank_for_schedule heuristics for SCHED_PRESSURE_MODEL.  SCHED_PRESSURE_MODEL is used for first scheduling pass in ARM backend by default.

The first one is when INSN_TICK of both instructions are equal, and rank_for_schedule returns a "tie" result, even though there are more heuristics down the path to break the tie.

The second one is to account for the fact that model_index() of two instructions is meaningful only when both instructions are in the current basic block.

Bootstrapped and tested on {x86_64,arm,aarch64}-linux.

OK to apply?

More fixes to SCHED_PRESSURE_MODEL coming soon.

Thank you,

--
Maxim Kuvyrkov
www.linaro.org

Attachment: 0001-Fix-INSN_TICK-heuristic-for-SCHED_PRESSURE.patch
Description: Binary data

Follow-Ups:
- Re: [PATCH] Fix INSN_TICK heuristic for SCHED_PRESSURE
  - From: Steven Bosscher

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]