On 26/11/15 16:49, Bernd Schmidt wrote:
On 11/26/2015 05:45 PM, Kyrill Tkachov wrote:
that doesn't help, punt. */
- modified_in_a = emit_a != NULL_RTX && modified_in_p (orig_b, emit_a);
if (tmp_b && then_bb)
{
These bits I thought would be part of a followup patch (which would also guard
against single_set problems), and as I mentioned I'd rather have a checking
assert.
Yes, you're right. I have the checking_assert statement in the followup that
I've been testing.
I'll move the deletion of these two statements there as well to minimise the
changes to this patch.
I'll move these bits to that patch, re-build cc1 and commit.
Thanks for your guidance,
Kyrill
So take these deletions out and leave them for the followup, and the patch is
ok everywhere. No need for a full retest given that practically the same patch
has been tested already, just make sure you can build cc1.
Bernd