[PATCH][IA64] Fix speculation support.
Vladimir Makarov
vmakarov@redhat.com
Tue Aug 5 15:39:00 GMT 2008
Maxim Kuvyrkov wrote:
> Hello,
>
> This patch fixes yet another issue of speculation support in
> scheduler. The bug is in initialization of new instructions emitted
> during scheduling: in some cases we initialize the data for new
> instructions twice, which causes sanity checks to fail.
>
> The particular case is that a new basic block with a single jump
> instruction is emitted at the end of the function to make a "pocket"
> for speculation recovery code. For that jump instruction we
> initialize dependency data structures twice: when emitting the
> instruction and later, when analyzing dependencies for that
> 1-instruction region.
>
> The patch cleans up extend_global() and extend_all() functions giving
> them better names and comments. All uses of these functions are
> updated to call the appropriate extender.
>
> I don't have access to ia64 box at the moment, so I only checked it on
> a cross-compiler. Andrey, can you, possibly, test the patch?
>
> OK for trunk and 4.3 branch?
>
The patch is OK. Thanks for the patch.
More information about the Gcc-patches
mailing list