[PATCH] D111117: [RISCV] Optimize (add (shl x, c0), c1)

[Ben Shi via Phabricator via llvm-commits]

benshi001 added a comment.

In D111117#3103171 <https://reviews.llvm.org/D111117#3103171>, @asb wrote:

> I'm getting a warning when building this (with clang 12.0.1):
>
>   /home/asb/llvm-project/llvm/lib/Target/RISCV/RISCVISelLowering.cpp:6490:20: warning: comparison of integers of different signs: 'int64_t' (aka 'long') and 'unsigned long' [-Wsign-compare]
>     if (C0 < 1 || C0 > VT.getSizeInBits() - 1 || isInt<12>(C1) ||

I have fixed in the newest patch.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111117/new/

https://reviews.llvm.org/D111117