1. C++ / Говнокод #7887

    +149

    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
    14. 14
    for(int i=0;i<size1;i+=4)
      {
        float4 boxMax(plist[i+0].Box().vmax[axis], plist[i+1].Box().vmax[axis], plist[i+2].Box().vmax[axis], plist[i+3].Box().vmax[axis]);
        float4 boxMin(plist[i+0].Box().vmin[axis], plist[i+1].Box().vmin[axis], plist[i+2].Box().vmin[axis], plist[i+3].Box().vmin[axis]);

        register __m128 a = _mm_and_ps(_mm_cmplt_ps(boxMax, vSplit), one);
        register __m128 b = _mm_and_ps(_mm_cmpgt_ps(boxMin, vSplit), one);
        
        register __m128 left  = _mm_sub_ps(one, b); // left  = 1 - b; same as not(b) 
        register __m128 right = _mm_sub_ps(one, a); // right = 1 - a; same as not(a) 

        summLeft += (left.m128_i32[0] + left.m128_i32[1] + left.m128_i32[2] + left.m128_i32[3]);
        summRight+= (right.m128_i32[0] + right.m128_i32[1] + right.m128_i32[2] + right.m128_i32[3]);
      }

    http://www.gamedev.ru/code/forum/?id=141592
    История про то, как Пятачок оптимизировал простой цикл:

    for(int i=0;i<plist.size();i++)
    
      {
    
        if(plist[i].Box().vmax[axis] < split)
    
          summLeft++;
    
        else if(plist[i].Box().vmin[axis] > split)
    
          summRight++;
    
        else
    
        {
    
          summLeft++;
    
          summRight++;
    
        }
    
      }

    Запостил: CPPGovno, 18 Сентября 2011

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

    • ava CPPGovno 18.09.2011 00:12 # 0

      >Жил был пятачок и написал он такой код:
      Ответить

    • ava 3.14159265 18.09.2011 08:52 # 0

      В чем говно-то?
      Использовал SSE - молодец.
      А насчет того, что оно тупит - нужно разобраться.
      Может unaligned обращения к памяти или еще какой нюанс.
      Ответить

      • ava CPPGovno 18.09.2011 09:21 # +1

        Да тут алгоритм нужно оптимизировать, а не оптимизировать простой цикл, переписывая на ассемблер.
        Ответить

        • ava Dummy00001 18.09.2011 12:24 # +1

          но ведь что бы переписать алгоритм, нужно думать!...
          Ответить

    • ava danilissimus 18.09.2011 11:31 # 0

      __m128? Какая архитектура?
      Ответить

    • ava guest 20.09.2011 00:18 # −1

      не говнокод, вообще не смешно. Возможно что алгоритм писал один разработчик, а машинную оптимизацию проводил другой. TC вероятно никогда не работал над большими в командах больше 10 разработчиков
      Ответить

      • ava CPPGovno 20.09.2011 06:01 # 0

        > Возможно что алгоритм писал один разработчик, а машинную оптимизацию проводил другой
        Пройдите по ссылке, если вам интересно. Это не так.
        Ответить

    • ava guest8 09.04.2019 11:31 # −999

      показать все, что скрытоvanished
      Ответить

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