Here is the updated patch with the #if defined(__has_attribute) && 
__has_attribute (target) guard. 

> Oh, you are right, sorry.

No worries! 😊

> Is the idea that we will put both "choose" functions in one file and the 
> actual
> CRC-32C code in another?  I'm okay with that.

Yup, there can only be one choose function that has to pick between avx512, 
sse42 and scalar code. 

Raghuveer

Attachment: v2-0001-Use-__attribute__-target-sse4.2-for-SSE42-CRC32C.patch
Description: v2-0001-Use-__attribute__-target-sse4.2-for-SSE42-CRC32C.patch

Reply via email to