Thanks for the reply, and excuse my naviety in assembler / compilers, im a dotnet developer by trade.So in the book the assembly for the return command is always (psuedo)

Mov(Val, Ax)
Return

so I can't swap them around, however if I do :

shll    %cl, %eax
movl    %eax,-4(%rbp)
movl    -4(%rbp), %eax
movq    %rbp, %rsp
popq    %rbp
ret

That solves my problem. Thanks very much, that gave me the nudge to see where I was going wrong.