[PATCH] D23930: [AArch64] Fix encoding for lsl #12 in add/sub immediates
Renato Golin via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 30 03:41:26 PDT 2016
rengolin added a comment.
In https://reviews.llvm.org/D23930#528639, @peter.smith wrote:
> From my reading of the code it seems like we'll fault this as out of range. If I'm right it would be nice to support this, but I don't think it necessarily needs to be done as part of this one.
Yes, this is an interesting enhancement, and should be reasonably simple. But not for this patch, I agree.
More information about the llvm-commits