lra: Stop eh_return data regs being incorrectly marked live [PR92989]
authorRichard Sandiford <richard.sandiford@arm.com>
Sat, 4 Apr 2020 16:23:40 +0000 (17:23 +0100)
committerRichard Sandiford <richard.sandiford@arm.com>
Mon, 6 Apr 2020 18:00:14 +0000 (19:00 +0100)
commite83714f65d1f75fc5af39f9fdc520a909dfc7635
tree6b068410184a628cc5c0e2400aebcf7487561721
parentb696698767ba45b4d61a93205167e2f1f744d3f1
lra: Stop eh_return data regs being incorrectly marked live [PR92989]

lra_assign has an assert to make sure that no pseudo is allocated
to a conflicting hard register.  It used to be restricted to
!flag_ipa_ra, but in g:a1e6ee38e708ef2bdef4 I'd enabled it for
flag_ipa_ra too.  It then tripped a few times while building
libstdc++ for mips-mti-linux.

Previous patches fixed one of the problems: registers clobbered
by the taking of an exception were being treated as live at the
beginning of the EH receiver, and this got propagated to predecessor
blocks.  But it turns out that there was a second problem: eh_return
data registers were also being marked live in the same way.

These registers are defined by the unwinder and so in reality they
are live on entry to the EH receiver.  But definitions can only happen
in blocks, not on edges, so for liveness purposes we use artificial
definitions at the start of the EH receiver.  process_bb_lives should
therefore model the effect of a definition, not a plain use.

2020-04-06  Richard Sandiford  <richard.sandiford@arm.com>

gcc/
PR rtl-optimization/92989
* lra-lives.c (process_bb_lives): Do not treat eh_return data
registers as being live at the beginning of the EH receiver.
gcc/ChangeLog
gcc/lra-lives.c