Opened 5 years ago

Last modified 7 months ago

#2725 new bug

Remove Hack in compiler/nativeGen/MachCodeGen.hs

Reported by: clemens Owned by: thoughtpolice
Priority: high Milestone: 7.8.3
Component: Compiler (NCG) Version: 6.11
Keywords: Cc: dterei, simonmar
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Difficulty: Unknown
Test Case: Blocked By:
Blocking: Related Tickets:

Description (last modified by igloo)

Remove the hack around Line 3914 labeled with:

    -- HACK: On x86_64 binutils<2.17 is only able to generate PC32
    -- relocations, hence we only get 32-bit offsets in the jump
    -- table. As these offsets are always negative we need to properly
    -- sign extend them to 64-bit. This hack should be removed in
    -- conjunction with the hack in PprMach.hs/pprDataItem once
    -- binutils 2.17 is standard.

This bug is intended for house keeping.

Change History (11)

comment:1 Changed 5 years ago by igloo

  • Description modified (diff)
  • Difficulty set to Unknown
  • Milestone set to 6.12 branch

comment:2 Changed 4 years ago by igloo

  • Milestone changed from 6.12 branch to 6.12.3

comment:3 Changed 4 years ago by igloo

  • Milestone changed from 6.12.3 to 6.14.1
  • Priority changed from normal to low

comment:4 Changed 3 years ago by igloo

  • Milestone changed from 7.0.1 to 7.0.2

comment:5 Changed 3 years ago by igloo

  • Milestone changed from 7.0.2 to 7.2.1

comment:6 Changed 3 years ago by dterei

  • Cc dterei added
  • Type of failure set to None/Unknown

comment:7 Changed 3 years ago by igloo

  • Milestone changed from 7.2.1 to 7.4.1

comment:8 Changed 2 years ago by igloo

  • Milestone changed from 7.4.1 to 7.6.1
  • Priority changed from low to lowest

comment:9 Changed 20 months ago by igloo

  • Milestone changed from 7.6.1 to 7.6.2

comment:10 Changed 17 months ago by igloo

  • Milestone changed from 7.6.2 to 7.8.1
  • Priority changed from lowest to high

This was opened 4 years ago; let's see if we can remove the hack and close it now.

nb, it's now in nativeGen/X86/CodeGen.hs.

comment:11 Changed 7 months ago by thoughtpolice

  • Cc simonmar added
  • Owner changed from clemens to thoughtpolice
Note: See TracTickets for help on using tickets.