search

intel / Immintrin-debug

immintrin_dbg.h is an include file, a wrapper around immintrin.h. It implements most of AVX, AVX2, AVX-512 vector intrinsics to enable source level debug of vector code.
BSD 3-Clause "New" or "Revised" License
57 stars 8 forks source link

UINT*_MAX should be INT*_MAX #8

Closed rosbif closed 4 years ago

rosbif commented 4 years ago

In the following functions both occurrences of UINT8_MAX should be INT8_MAX:- Saturate_UnsignedInt16_To_Int8() Saturate_UnsignedInt32_To_Int8() Saturate_UnsignedInt64_To_Int8()

In the following functions both occurrences of UINT16_MAX should be INT16_MAX:- Saturate_UnsignedInt64_To_Int16() Saturate_UnsignedInt32_To_Int16()

In the following function both occurrences of UINT32_MAX should be INT32_MAX:- Saturate_UnsignedInt64_To_Int32()

rosbif commented 4 years ago

You also modified the one in Saturate_To_UnsignedInt16() on line 117. But this one was correct and should be UINT16_MAX as before.

rosbif commented 4 years ago

You must change both INT16_MAX on line 117 back to UINT16_MAX. You only changed the second one.