Re: [PATCH v3 5/5] net/mlx5: add vectorized Rx/Tx burst for SSE4.1

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 7/5/2017 7:12 PM, Yongseok Koh wrote:
> To make vectorized burst routines enabled, it is required to run on x86_64
> architecture which can support at least SSE4.1. If all the conditions are
> met, the vectorized burst functions are enabled automatically. The decision
> is made individually on RX and TX. There's no PMD option to make a
> selection.
> 
> Signed-off-by: Yongseok Koh <yskoh@mellanox.com>

Hi Yongseok,

clang [2] is producing following build error [1], can you please check?

Thanks,
ferruh


[1]
.../dpdk/drivers/net/mlx5/mlx5_rxtx_vec_sse.c:686:23: error: cannot
convert between vector values of different size ('__m128i' (vector of 2
'long long' values) and 'unsigned long')
                invalid_mask = zero | ((mcqe_n - pos) * sizeof(uint16_t)
* 8);
                               ~~~~ ^
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
.../dpdk/drivers/net/mlx5/mlx5_rxtx_vec_sse.c:1055:15: error: cannot
convert between vector values of different size ('__m128i' (vector of 2
'long long' values) and 'unsigned long')
                mask = zero | ((pkts_n - pos) * sizeof(uint16_t) * 8);
                       ~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
.../dpdk/drivers/net/mlx5/mlx5_rxtx_vec_sse.c:1163:15: error: cannot
convert between vector values of different size ('__m128i' (vector of 2
'long long' values) and 'unsigned long')
                mask = zero | (comp_idx * sizeof(uint16_t) * 8);
                       ~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
.../dpdk/drivers/net/mlx5/mlx5_rxtx_vec_sse.c:1172:15: error: cannot
convert between vector values of different size ('__m128i' (vector of 2
'long long' values) and 'unsigned long')
                mask = zero | (n * sizeof(uint16_t) * 8);
                       ~~~~ ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~
4 errors generated.

[2]
ICC is giving build error for same lines, I assume fixing clang will fix
the ICC,