__builtin_bswap16

может просто

#define bswap16(x) ((unsigned short)(x)<<8 | (unsigned short)(x)>>8)

может просто

Не-не-не. Если бы чисто моё было, тогда да. Но нужен именно __builtin_bswap16(x).

ну добавь касты в unsigned (ибо двигать signed нехорошо, а bswap там делать можно)

ну добавь касты в unsigned

Расшифруй. Не всёк.

Сдвигать отрицательное число - UB.

Сдвигать отрицательное число - UB.

Знаю, знаю. Но не предвидится. Так то криво, согласен. Но не предвидится. Хотя, чорт его знает.

#define bswap16(x) ((unsigned short)(x)<<8 | (unsigned short)(x)>>8)

Не, плохая идея. У тебя это может сломаться, если кто-то сделает

bswap16(x++)

тогда функцией надо. static inline например

#if GCC_VERSION < 480

Меня больше это волнует. Разве норм?

__builtin_bswap16 не макрос, потому не знаю, есть ли другие способы определить
Ну можно в конце концов __GNUC__ и __GNUC_MINOR__ напрямую проверять. строчка получится длиннее

ну в моём варианте если не gcc оно будет 0

ну в моём варианте если не gcc оно будет 0

Вот именно, вот именно. В троём компиляторе __builtin_bswap16 определён?

__builtin_bswap16 определен в GCC и мимикрирующих под него компиляторах (clang и Intel С Сompiler под линуксом). В древних версиях GCC https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52624 его например нет под некоторые платформы

ну в моём варианте если не gcc оно будет 0

Может всё-таки?:

#if GCC_VERSION < 480 && GCC_VERSION > 0

В древних версиях GCC его например нет под некоторые платформы

Из-за этого и сыр-бор.

И меня беспокоит наличие GCC_VERSION в случаях, когда он должен быть undef. Вот это мне не нравится.

а откуда оно будет на не-gcc?

а откуда оно будет на не-gcc?

Ну у тебя же получилось 0. Что-то здесь явно не так.

Ну так #if defined(__GNUC__) проверяет и выставляет там 0.

#if defined(__GNUC__)
#define GCC_VERSION (__GNUC__ * 100 + __GNUC_MINOR__ * 10)
#else
#define GCC_VERSION 0 <----

Ну так #if defined(__GNUC__) проверяет и выставляет там 0.

Промежуточный итог. Новая точка отсчёта, так сказать:

#if defined(__GNUC__)
 #define GCC_VERSION (__GNUC__ * 100 + __GNUC_MINOR__ * 10)
 #if GCC_VERSION < 480
  #define bswap16(x) (((unsigned short)(x))<<8 | ((unsigned short)(x))>>8)
  #define __builtin_bswap16(x) bswap16((unsigned short)(x))
 #endif
 #undef GCC_VERSION
#endif

Прокатит?

Прокатит?

Нет. Делай inline функцию для bswap16 которая б принимала uint16_t и возвращала uint16_t, заверни ее тоже в ifdef. По поводу проблемы с макросом bswap16 я написал в __builtin_bswap16 (комментарий)

По поводу проблемы с макросом bswap16 я написал в __builtin_bswap16 (комментарий)

Так из-за этого два определения и сделал, вместо одного. Не прокатит?

Нет, не прокатит

https://godbolt.org/z/1WiMkz

<source>: In function 'main':
<source>:1:51: warning: operation on 'a' may be undefined [-Wsequence-point]
   #define bswap16(x) (((unsigned short)(x))<<8 | ((unsigned short)(x))>>8)
                                                  ~^~~~~~~~~~~~~~~~~~~~
<source>:2:33: note: in expansion of macro 'bswap16'
   #define m__builtin_bswap16(x) bswap16((unsigned short)(x))
                                 ^~~~~~~
<source>:7:22: note: in expansion of macro 'm__builtin_bswap16'
   unsigned short b = m__builtin_bswap16(a++);
                      ^~~~~~~~~~~~~~~~~~
Compiler returned: 0

Нет, не прокатит

Печалька. Лишнее получается. Ну, значит static inline. А с ним, если signed, только варнинг будет?

Нет, не должно там быть никаких варнингов. Хотя может и есть какие-то компиляторы или опции компилятора, при которых компилятор может на это кидать варнинг, я хз. Можешь кастовать макросом, если тебе это прям так важно

Нет, не должно там быть никаких варнингов.

Хорошо. Ну, последняя редакция. Я надеюсь:

#if defined(__GNUC__)
 #define GCC_VERSION (__GNUC__ * 100 + __GNUC_MINOR__ * 10)
 #if GCC_VERSION < 480
  static inline unsigned short __builtin_bswap16(unsigned short x)
  {
    return (x<<8)|(x>>8);
  }
 #endif
 #undef GCC_VERSION
#endif

а зачем вообще определять макрос GCC_VERSION а потом его undef делать? Ну как минимум он может быть уже определен. Можно же написать

#if defined(__GNUC__)
  #if (__GNUC__ < 4) || ((__GNUC__ == 4) && (__GNUC_MINOR__ < 8))
  static inline unsigned short __builtin_bswap16(unsigned short x)
  {
    return (x<<8)|(x>>8);
  }
 #endif /* GCC < 4.8.0 */
#endif /* __GNUC__ */

И наверное стоит использовать uint16_t вместо unsigned short

а зачем вообще определять макрос GCC_VERSION

Это притянулось из исходного варианта. И именно это меня смущало.

И наверное стоит использовать uint16_t вместо unsigned short

По-моему, разницы нет.

И наверное стоит использовать uint16_t вместо unsigned short
По-моему, разницы нет.

для x86 нет. если не будет кросс-компиляции (где могут быть другие размеры типов данных), то это не нужно.

Вот это смотрели? https://github.com/nemequ/portable-snippets/blob/master/builtin/builtin.h

Вот это смотрели?

Смотрели. Но за ссылку всё-равно сенькс, сам забыл втемяшить. Интересует всё-таки только gcc.

если не будет кросс-компиляции

Твою ж мать. Согласен.

asm volatile(".byte 0x66; bswap %%eax;":«=a»(out):«a»(in));

юзай

[code\]

Юзай [[inline]]

Интересует всё-таки только gcc

Тогда непонятно, в чём проблема с #if GCC_VERSION ?

Тогда непонятно, в чём проблема с #if GCC_VERSION ?

Лишнее определение.