Opened 3 years ago

Closed 9 months ago

Last modified 9 months ago

#5308 closed feature request (fixed)

Generalize -msse2 command line flag to -msse<version number>

Reported by: tibbe Owned by:
Priority: low Milestone: 7.6.2
Component: Compiler Version: 7.0.4
Keywords: Cc: johan.tibell@…
Operating System: Unknown/Multiple Architecture: Unknown/Multiple
Type of failure: None/Unknown Difficulty: Unknown
Test Case: Blocked By:
Blocking: Related Tickets:

Description

I'm looking into adding support for the popcnt instruction. The instruction should be used if the user specifies -msse4.2*. I think we should generalize the -msse2 flag to be just -msse followed by a version number.

  • Although not formally part of the SSE 4.2 instruction set, the popcnt instruction was added at the same time. GCC enables the popcnt instruction if you specify -msse4.2, so I thought it'd be reasonable for us to do so as well.

Change History (8)

comment:1 Changed 3 years ago by igloo

  • Milestone set to 7.4.1

comment:2 Changed 2 years ago by igloo

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

comment:3 Changed 19 months ago by igloo

  • Milestone changed from 7.6.1 to 7.6.2

comment:4 Changed 9 months ago by tibbe

  • Difficulty set to Unknown
  • Resolution set to fixed
  • Status changed from new to closed

I fixed this at some point. I don't have the commit handy.

comment:5 Changed 9 months ago by carter

do we also have -mavx /-mavx2 prebaked or are we going to need that added later? I'm currently doing the footwork to add support for avx registers to the ghc calling convention

comment:6 Changed 9 months ago by tibbe

We don't have -mavx/-mavx2.

comment:7 Changed 9 months ago by carter

ok, once i've gotten the augmented calling convention stuff ok'd for llvm, i'll look into that

comment:8 Changed 9 months ago by gmainland

I added support for -mavx* on the simd branch. Could be cherry-picked after review.

Note: See TracTickets for help on using tickets.