Hi Thomas ,

Pls reject it 

-----Original Message-----
From: Thomas Monjalon <tho...@monjalon.net> 
Sent: Friday, April 9, 2021 4:12 AM
To: Amber, Kumar <kumar.am...@intel.com>
Cc: Wang, Yipeng1 <yipeng1.w...@intel.com>; dev@dpdk.org; Richardson, Bruce 
<bruce.richard...@intel.com>; Gobriel, Sameh <sameh.gobr...@intel.com>
Subject: Re: [dpdk-dev] [PATCH v1] lib/hash: support non sse42 cpu architecture

25/03/2021 09:06, Thomas Monjalon:
> 24/03/2021 23:59, Wang, Yipeng1:
> > From: kumar amber
> > > 
> > > add _SSE42_ flag to enable compilation of
> > > sse42 specific instructions only on supported architecture
> > > 
> > > Signed-off-by: kumar amber <kumar.am...@intel.com>
[...]
> > [Wang, Yipeng] 
> > Hi, Kumar, thanks for the patch.
> > I think the minimum required machine for x86 is sse4.2 compatible already. 
> > So I wonder if we really need this.
> 
> Yes, that's why I don't understand this patch.

Any comment? Should we reject the patch?



Reply via email to