2016-05-20 7 views
9

Я пробую векторизовать следующую функцию с clang в соответствии с этим clang reference. Он принимает вектор байтового массива и применяет маску в соответствии с this RFC.Векторизовать функцию в clang

static void apply_mask(vector<uint8_t> &payload, uint8_t (&masking_key)[4]) { 
    #pragma clang loop vectorize(enable) interleave(enable) 
    for (size_t i = 0; i < payload.size(); i++) { 
    payload[i] = payload[i]^masking_key[i % 4]; 
    } 
} 

Следующие флаги передаются лязг:

-O3 
-Rpass=loop-vectorize 
-Rpass-analysis=loop-vectorize 

Однако векторизации выдает следующее сообщение об ошибке:

WebSocket.cpp:5: 
WebSocket.h:14: 
In file included from boost/asio/io_service.hpp:767: 
In file included from boost/asio/impl/io_service.hpp:19: 
In file included from boost/asio/detail/service_registry.hpp:143: 
In file included from boost/asio/detail/impl/service_registry.ipp:19: 
c++/v1/vector:1498:18: remark: loop not vectorized: could not determine number 
     of loop iterations [-Rpass-analysis] 
    return this->__begin_[__n]; 
       ^
c++/v1/vector:1498:18: error: loop not vectorized: failed explicitly specified 
     loop vectorization [-Werror,-Wpass-failed] 

Как векторизации этот цикл?

+0

Что произойдет, если вы захватываете размер вектора вне цикла и использовать его для условия цикла? – NathanOliver

+0

Пробовал это уже. Не помогает! – rahul

+1

Эта петля выглядит тривиальной для векторизации. Вы проверили, не делает ли это компилятор неявным с помощью простого '-03'? –

ответ

4

Благодаря @PaulR и @PeterCordes. Развертывание цикла выполняется в 4 раза.

void apply_mask(vector<uint8_t> &payload, const uint8_t (&masking_key)[4]) { 
    const size_t size = payload.size(); 
    const size_t size4 = size/4; 
    size_t i = 0; 
    uint8_t *p = &payload[0]; 
    uint32_t *p32 = reinterpret_cast<uint32_t *>(p); 
    const uint32_t m = *reinterpret_cast<const uint32_t *>(&masking_key[0]); 

#pragma clang loop vectorize(enable) interleave(enable) 
    for (i = 0; i < size4; i++) { 
    p32[i] = p32[i]^m; 
    } 

    for (i = (size4*4); i < size; i++) { 
    p[i] = p[i]^masking_key[i % 4]; 
    } 
} 

gcc.godbolt code

+1

Это была идея Павла (https://stackoverflow.com/questions/37351236/vectorize-a-function-in-clang#comment62218869_37351236). :) –

+2

Unrolling - это единственный способ получить его векторизованный, я пробовал все компиляторы на gcc.godbolt, и никто не смог векторизовать без разворота. – GameDeveloper

+1

@BaummitAugen Спасибо, что указали это. Исправлена. – rahul

Смежные вопросы