2016-05-20 17 views
9

Próbuję wektoryzować następującą funkcję z klang zgodnie z tym clang reference. Pobiera wektor tablicy bajtów i stosuje maskę zgodnie z this RFC.Wektoryzacja funkcji w klang

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]; 
    } 
} 

następujące flagi są przekazywane do dzyń:

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

Jednak wektoryzacja nie powiedzie się z powodu następującego błędu:

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] 

Jak wektorować to dla pętli?

+0

Co się dzieje, jeśli uchwycić wielkość wektora zewnątrz pętli i użyć jej do stanu pętli? – NathanOliver

+0

Wypróbowałem już. Nie pomaga! – rahul

+1

Ta pętla wygląda na trywialną, aby wektoryzować. Czy sprawdziłeś, czy kompilator robi to niejawnie z zwykłym '-03'? –

Odpowiedz

4

Dzięki @PaulR i @PeterCordes. Rozwinięcie pętli czterokrotnie działa.

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

To był [pomysł Pawła] (https://stackoverflow.com/questions/37351236/vectorize-a-function-in-clang#comment62218869_37351236). :) –

+2

Rozwinięcie jest jedynym sposobem, aby uzyskać wektoryzacji, próbowałem na wszystkich kompilatorach na gcc.godbolt i nikt nie był w stanie wektoryzacji bez rozwijania. – GameDeveloper

+1

@BaummitAugen Dzięki za wskazanie tego. Naprawiony. – rahul