Ticket #2045 (closed merge: fixed)

Opened 5 years ago

Last modified 5 years ago

Link error when compiling with -fhpc

Reported by: guest Owned by: igloo
Priority: normal Milestone: 6.10.1
Component: Compiler Version: 6.8.2
Keywords: Cc:
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: Difficulty: Unknown
Test Case: T2045 Blocked By:
Blocking: Related Tickets:

Description

I've attached a version of Lava and simple program Main.hs. Compiling without -fhpc works fine, but with -fhpc, gives a link error (Neil says it's trying to find the 74 element tupe and can't).

Attachments

hpcbug.tar.gz Download (75.9 KB) - added by guest 5 years ago.
Vhdl.hs Download (4.6 KB) - added by igloo 5 years ago.

Change History

Changed 5 years ago by guest

Changed 5 years ago by guest

Forgot to pase the exact error message:

Linking gencirc ... ../Lava2000/Modules/Vhdl.o: In function `s13oq_info': (.text+0xbd5): undefined reference to `base_DataziTuple_Z74T_con_info' collect2: ld returned 1 exit status

Changed 5 years ago by igloo

  • difficulty set to Unknown
  • milestone set to 6.8.3

Thanks for the report. I don't know how easy this will be to fix, but let's put it in the 6.8.3 milestone for now.

Changed 5 years ago by igloo

  • owner set to igloo

Changed 5 years ago by igloo

Changed 5 years ago by igloo

  • owner igloo deleted

Vhdl.hs is a small, self-contained example. Compile with:

ghc --make Vhdl.hs -o gencirc

As I understand it GHC is supposed to generate large tuples and then have them disappear later on. My guess is that the HPC ticks are interfering with the elimination, but I'm not sure where that is supposed to happen.

Changed 5 years ago by simonpj

  • owner set to igloo
  • testcase set to T2045
  • type changed from bug to merge

Fixed by

  * Fix Trac #2045: use big-tuple machiney for implication constraints

    M ./compiler/deSugar/DsUtils.lhs -5 +5
    M ./compiler/typecheck/TcSimplify.lhs -8 +8

Might be worth trying to merge this to the 6.8 branch

Simon

Changed 5 years ago by simonpj

Oops: you also need this patch too, sorry

Thu Jun  5 17:54:34 BST 2008  simonpj@microsoft.com
  * Vital follow-up to fix of Trac #2045

Changed 5 years ago by igloo

  • status changed from new to closed
  • resolution set to fixed
  • milestone changed from 6.8.3 to 6.10.1

Changed 5 years ago by simonmar

  • architecture changed from Unknown to Unknown/Multiple

Changed 5 years ago by simonmar

  • os changed from Unknown to Unknown/Multiple
Note: See TracTickets for help on using tickets.