Ticket #7585 (closed bug: fixed)

Opened 4 months ago

Last modified 4 months ago

Core lint failure when optimizing coercions in branched axioms

Reported by: goldfire Owned by: goldfire
Priority: normal Milestone:
Component: Compiler Version: 7.7
Keywords: TypeFamilies Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: Compile-time crash Difficulty:
Test Case: Blocked By:
Blocking: Related Tickets:

Description

The attached code causes the failure.

Core Lint correctly checks branched axioms for internal consistency -- when using branch n of an axiom, we must ensure that no branch m < n can possibly apply, no matter what the instantiation for any type variables in the branch may be. However, the coercion optimizer does not respect this property. It will replace coercions used in axioms with equivalent coercions that do not respect this internal consistency property. Everything works out OK in the end (without -dcore-lint, the file compiles and runs correctly), but we go through an invalid state on the way.

In particular, the TrPushAx rules are to blame.

I'm not sure what the best fix for this is, but it seems to be my job to find it.

Attachments

Bug.hs Download (0.5 KB) - added by goldfire 4 months ago.
triggers the bug

Change History

Changed 4 months ago by goldfire

triggers the bug

Changed 4 months ago by eir@…

commit a6ab0a40ac750914640613c77148e948fabf35d5

Author: Richard Eisenberg <eir@cis.upenn.edu>
Date:   Tue Jan 15 17:19:37 2013 -0500

    Fix Trac #7585.
    
    The coercion optimizer was optimizing coercions inside of branched
    axiom applications, sometimes invalidating the branch choice within
    the axiom application. Now, we check to make sure we are not
    invalidating this invariant before proceeding with the optimization.

 compiler/coreSyn/CoreLint.lhs  |   41 +------------------
 compiler/types/OptCoercion.lhs |   88 +++++++++++++++++++++++++++++++++++-----
 2 files changed, 79 insertions(+), 50 deletions(-)

Changed 4 months ago by goldfire

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.