GmSSL-master\include\gmssl\endian.h 中PUTU64宏的使用存在bug

[ckooying]

昨天在项目中使用sm4_gcm解密数据时发现比Aes_gcm慢很多，通过跟踪分析发现在文件GmSSL-master\src\gf128.c中gf128_to_bytes函数里的PUTU64宏的使用很可疑，其里面包含了对reverse_bits函数调用，按理每次PUTU64应该只调用一次reverse_bits，但实际日志发现每次PUTU64调用了8次reverse_bits，而reverse_bits是对64bit数据的反转，后通过将 reverse_bits(a[0]) 移出PUTU64宏调用，程序性能立即变好了，sm4_gcm和Aes_gcm不相上下，故确认宏的这种使用存在问题，编译器没有优化成只进行一次调用，而是8次调用。
原有代码如下：
void gf128_to_bytes(const gf128_t a, uint8_t p[16])
{
PUTU64(p, reverse_bits(a[0]));
PUTU64(p + 8, reverse_bits(a[1]));
}
修改代码如下：
void gf128_to_bytes(const gf128_t a, uint8_t p[16])
{
uint64_t a0=reverse_bits(a[0]);
PUTU64(p, a0);
uint64_t a1=reverse_bits(a[1]);
PUTU64(p + 8, a1);
}
具体测试数据见附件