[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.


https://reviews.llvm.org/D23930





More information about the llvm-commits mailing list