[Bug target/44987] New: *mmintrin.h files incompatible with partial __attribute__((target("sse...")))

[Bug target/44987] New: *mmintrin.h files incompatible with partial __attribute__((target("sse...")))

[andi-gcc at firstfloor dot org]

The *mmintr.h files have checks like

#ifndef __SSE4_2__
# error "SSE4.2 instruction set not enabled"
#else

Problem with this is that it assumes that the whole file is compiled with 
-msse4.2 or similar option.

But with __attribute__((target(...))) it's possible and appropiate to 
let only part of the code use e.g. -msse4.2 and compile the rest
of the file with different options.

The problem is just that *mmintrin.h cannot be used then because 
__SSE4_2__ or similar is not set by the target attribute.

One workaround I found was to specify all the defines manually before
including the intrin.h header. But that's rather ugly because it needs
7 different defines or so.

I think it would be better to drop these #ifdefs from the headers
and check the code generation settings for the current function in
the intrinsics instead.


-- 
           Summary: *mmintrin.h files incompatible with partial
                    __attribute__((target("sse...")))
           Product: gcc
           Version: 4.6.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: andi-gcc at firstfloor dot org
  GCC host triplet: x86_64-linux


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44987

[Bug target/44987] *mmintrin.h files incompatible with partial __attribute__((target("sse...")))

[rguenth at gcc dot gnu dot org]

------- Comment #1 from rguenth at gcc dot gnu dot org  2010-07-19 13:03 -------
Confirmed.


-- 

rguenth at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2010-07-19 13:03:30
               date|                            |


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44987

[Bug target/44987] *mmintrin.h files incompatible with partial __attribute__((target("sse...")))

[hjl dot tools at gmail dot com]

------- Comment #2 from hjl dot tools at gmail dot com  2010-07-19 13:27 -------
The design of target attribute is incomplete and
its implementation is broken. See PR 37565.

I'd like to see a new design/implementation. We
should deal with all issues, including header files,
in the new design/implementation.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44987

[Bug target/44987] *mmintrin.h files incompatible with partial __attribute__((target("sse...")))

[andi-gcc at firstfloor dot org]

------- Comment #3 from andi-gcc at firstfloor dot org  2010-07-19 15:47 -------
HJ, are you saying that target("sse4.2") is broken too (seems to work here in
simple tests?) or just that some target sub options are broken?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44987

[Bug target/44987] *mmintrin.h files incompatible with partial __attribute__((target("sse...")))

[hjl dot tools at gmail dot com]

------- Comment #4 from hjl dot tools at gmail dot com  2010-07-19 18:01 -------
For the time being, I suggest you compile it with
a different file. One approach is to implement
something similar to cpu dispatcher in icc via
GNU_IFUNC.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44987

[Bug target/44987] *mmintrin.h files incompatible with partial __attribute__((target("sse...")))

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44987

Andrew Pinski <pinskia at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |4.9.0
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #5 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Fixed since r0-124016 for GCC 4.9.0