Fix MaybeAlign -> Align

cdl-saarland / rv

RV: A Unified Region Vectorizer for LLVM

Other

105 stars 15 forks source link

Fix MaybeAlign -> Align #43

Closed baziotis closed 4 years ago

baziotis commented 4 years ago

test_067_recurr-loop.cpp fails without logs.

simoll commented 4 years ago

Thx for the patch!

test_067_recurr-loop.cpp fails without logs.

If they do not map to strided vector shapes, we do not currently handle recurrences (non-reduction header phi nodes) in the loop being vectorized, ie:

  LoopHeader:
    %x = phi [%x.next, LoopLatch], ...
    Some Use Of %x
    ; ...
    %x.next = ...
    ; ...
    br LoopHeader

The (yet unreleased) loop distribution branch is aiming to solve that in a general way.

baziotis commented 4 years ago

Ok, thanks. I thought that it was my patch that broke it.