This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: Support new PNI intrinsics
On Tue, Jun 24, 2003 at 10:10:29AM -0700, Richard Henderson wrote:
> On Fri, Jun 20, 2003 at 05:00:56PM -0700, H. J. Lu wrote:
> > -#define MASK_SSE1 (MASK_SSE | MASK_SSE2)
> > -#define MASK_SSE164 (MASK_SSE | MASK_SSE2 | MASK_64BIT)
> > +#define MASK_SSE_1 (MASK_SSE | MASK_SSE2 | MASK_PNI)
> > +#define MASK_SSE_2 (MASK_SSE2 | MASK_PNI)
> > +#define MASK_SSE_164 (MASK_SSE | MASK_SSE2 | MASK_64BIT)
> > #define MASK_SSE264 (MASK_SSE2 | MASK_64BIT)
>
> Why did you rename these, or worse *some* of these?
> Seems to me all it does it bulk up the patch and
> obscure whatever else is going on.
The old MASK_SSE1 allows -msse2 to enable SSE. The new -mpni should
enable SSE and SSE2. Unfortunately, Unlike MASK_SSE1, which is only
usd to enable SSE, MASK_SSE2 is used to both select and enable SSE2.
I couldn't find a way to enable SSE2 by -mpni without introducing
a new macro, MASK_SSE_2, which is only used to enable SSE2. What is
the best way to achieve that?
H.J.