1. Си / Говнокод #20090

    −48

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    # define l2cn(l1,l2,c,n) { \
                        c+=n; \
                        switch (n) { \
                        case 8: *(--(c))=(unsigned char)(((l2)>>24L)&0xff); \
                        case 7: *(--(c))=(unsigned char)(((l2)>>16L)&0xff); \
                        case 6: *(--(c))=(unsigned char)(((l2)>> 8L)&0xff); \
                        case 5: *(--(c))=(unsigned char)(((l2)     )&0xff); \
                        case 4: *(--(c))=(unsigned char)(((l1)>>24L)&0xff); \
                        case 3: *(--(c))=(unsigned char)(((l1)>>16L)&0xff); \
                        case 2: *(--(c))=(unsigned char)(((l1)>> 8L)&0xff); \
                        case 1: *(--(c))=(unsigned char)(((l1)     )&0xff); \
                                } \
                        }

    https://github.com/openssl/openssl/blob/6218a1f57e7e25a6b9a798f00cf5f0e56a02ff31/crypto/des/des_locl.h#L151
    Интересно, на кой им делать там &0xff если оно в unsigned char скастовано и лишние биты там уже обрезаны? Или они там предполагают что unsigned char может быть не 8-битным? И вообще, это какая-то херня, почему б просто не узнать endian и не сделать memcpy предварительно развернув байты в случае необходимости? Другого макроговна там тоже хватает

    Запостил: j123123, 28 Мая 2016

    Комментарии (24) RSS

    • ava j123123 28.05.2016 10:57 # +2 

      # elif defined(__GNUC__) && __GNUC__>=2 && !defined(__STRICT_ANSI__) && !defined(OPENSSL_NO_ASM) && !defined(OPENSSL_NO_INLINE_ASM) && !defined(PEDANTIC)
#  if defined(__i386) || defined(__i386__) || defined(__x86_64) || defined(__x86_64__)
#   define ROTATE(a,n)   ({ register unsigned int ret;   \
                                asm ("rorl %1,%0"       \
                                        : "=r"(ret)     \
                                        : "I"(n),"0"(a) \
                                        : "cc");        \
                           ret;                         \
                        })

      Похоже что они не очень хорошего мнения об оптимизаторах gcc и clang раз считают нужным писать такие асмовставки
      Ответить

      • ava j123123 28.05.2016 11:31 # 0

        к тому же в gcc(да и в clang тоже, они там все за gcc повторяют зачем-то) для этого есть интринсик
        Ответить

        • ava guesto 28.05.2016 12:01 # 0

          Хотелось быиузнать в асм какой машины интрисит шланг
          Ответить

          • ava bormand 28.05.2016 12:13 # 0

            Низкоуровневой виртуальной?
            Ответить

            • ava guesto 28.05.2016 13:38 # 0

              Правильно. Для ллвм. А оттуда ее бекенд делает асм под арм интел итд. Так в каком месте появляется интринсик?
              Ответить

              • ava bormand 28.05.2016 13:40 # 0

                Дык интринсик в обход си, которое не позволяет по-человечески выразить все эти rol'ы. В самой llvm эти команды, наверное, есть.
                Ответить

                • ava guesto 28.05.2016 13:41 # 0

                  А в ллвм есть рорл?
                  Ответить

                  • ava bormand 28.05.2016 13:43 # 0

                    Хуй знает. Нету?
                    Ответить

                    • ava guesto 28.05.2016 13:46 # 0

                      Не знаю. Мне просто интересно как протащить интринсик через 2 уровня абстракции. В жабе тоше есть интринсики правда, но там о них jvm знает
                      Ответить

                      • ava bormand 28.05.2016 13:47 # 0

                        Ну если llvm позволяет добавлять кастомные инструкции - можно. Но тогда получается неявная договорённость между компилятором и генератором нативного кода. Плюс получится IR, который понимают только они, что портит всю идею llvm...
                        Ответить

        • ava Bobik 28.05.2016 14:48 # 0

          ЕМНИП, интрисинк для поворотов есть только в студии, а в остальных есть оптимизация двух сдвигов и | до одной инструкции.
          Ответить

          • ava j123123 28.05.2016 15:03 # 0 

            https://gcc.gnu.org/viewcvs/gcc/trunk/gcc/config/i386/ia32intrin.h?revision=232055&view=markup#l129

            У шланга тоже где-то это должно быть
            Ответить

            • ava j123123 28.05.2016 15:19 # 0

              Хотя похоже что нету там этого. Ну и пофиг
              https://groups.google.com/forum/#!topic/llvm-dev/9ZreOEi2znI
              Ответить

            • ava Bobik 28.05.2016 15:29 # 0

              У GCC в этом хедере они тоже написаны через сдвиг для 32 и 64 битных чисел.
              Ответить

    • ava gost 28.05.2016 13:48 # +1

      > l2cn
      Какие охуительные названия макросов. Интересно, почему большинство людей, дорвавшись до препроцессора, начинают аццки экономить байты исходников, даже если до этого были вполне вменяемыми остальным сущностям дают чёткие и понятные имена?..
      Ответить

      • ava guesto 28.05.2016 13:50 # +4

        Прст зчм лшне бкв пст? И тк вс пнтн
        Ответить

      • ava bormand 28.05.2016 13:51 # +7

        Всю жизнь сишника ебали и били палкой за короткие имена, макросы и битоёбство без повода. Но вот, в один прекрасный день, какой-то фрагмент кода всё-таки упёрся в производительность. И сишник на пару дней обрёл свободу...

        Завтра его снова будут ебать и бить палкой. Но сегодня ему можно всё.
        Ответить

        • ava guesto 28.05.2016 13:55 # 0

          Строго говоря в анси и позикс его соооовсем не били за короткие имена
          Ответить

        • ava gost 28.05.2016 14:15 # +2

          Прямо как аннотация для нового голливудского блокбастера! Я б посмотрел.
          Ответить

    • ava Soul_re@ver 28.05.2016 16:50 # 0

      >Или они там предполагают что unsigned char может быть не 8-битным?
      Если это код для каких-нибудь сраных контроллеров, то может. Но в таком случае возникает вопрос о типе "c" и поведении этого кода. Плюс на портабельность в таком случае давно положен хуй и можно пользоваться знаниями о конкретной архитектуре.
      Ответить

    Добавить комментарий