On Sun, Dec 22, 2013 at 10:52 PM, Andrey Belevantsev <a...@ispras.ru> wrote:
> Hello,
>
> As described in the PR, the ICE reason was the typo made when introducing
> calls to add_hard_reg_set.  Fixed by the first attached patch, bootstrapped
> and tested on both ia64 and x86_64, committed as obvious.
>
> The test case is very sensitive to the scheduler decisions (e.g. it didn't
> fail on trunk but only on the revision reported for me), so instead of
> adding the test I have put in the code two asserts checking that we can
> always schedule the fence instruction as is.  This hunk was tested together
> with the first but committed separately.
>

Testcase is very small. Why not add it?


-- 
H.J.

Reply via email to