[LLVMbugs] [Bug 4226] ARM does not lower all vshifts

[bugzilla-daemon at cs.uiuc.edu]

http://llvm.org/bugs/show_bug.cgi?id=4226


Xerxes Rånby <xerxes at zafena.se> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED




--- Comment #14 from Xerxes Rånby <xerxes at zafena.se>  2009-08-25 02:50:32 ---
(In reply to comment #13)
> (In reply to comment #12)
> > Also, it's got to be said that TLI.getOperationAction
> > is overloaded: it's used for results and operands and
> > that can cause confusion like here.
> 
> Mmm... have any suggestions for improvements?  It'll be significantly easier to
> make non-trivial changes now that LegalizeDAG is mostly cleaned up.
> 

After your changes in llvm rev 79719 the ARM target now passes the CodeGen
test!
http://llvm.org/viewvc/llvm-project?view=rev&revision=79719

xerxes at sheeva:/usr/src/build/buildbot/llvm-arm-linux/build$ ./Debug/bin/llvm-as
< test/CodeGen/ARM/vshift_split.ll | ./Debug/bin/llc -march=arm
        .file   "<stdin>"
        .eabi_attribute 20, 1
        .eabi_attribute 21, 1
        .eabi_attribute 23, 3
        .eabi_attribute 24, 1
        .eabi_attribute 25, 1


        .text
        .globl  update
        .align  2
update:
        mov r0, r0, lsr #2
        orr r0, r0, r1, lsl #30
        mov r2, r2, lsr #2
        orr r2, r2, r3, lsl #30
        mov r1, r1, lsr #2
        mov r3, r3, lsr #2
        bx lr
        .size update, .-update


Thanks!
Bug closed


-- 
Configure bugmail: http://llvm.org/bugs/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.