[AVR] Remove unsigned <= 0 checks. NFCI.
authorSimon Pilgrim <llvm-dev@redking.me.uk>
Fri, 22 May 2020 11:28:26 +0000 (12:28 +0100)
committerSimon Pilgrim <llvm-dev@redking.me.uk>
Fri, 22 May 2020 11:28:39 +0000 (12:28 +0100)
D77207 changed the bounds checks resulting in tests for positive unsigned values - dropping the superfluous check to fix gcc+Werror "error: comparison of unsigned expression >= 0 is always true [-Werror=type-limits]" warning.

llvm/lib/Target/AVR/AVRInstrInfo.td

index 5012ddf..e64f465 100644 (file)
@@ -128,21 +128,21 @@ def ioaddr8 : PatLeaf<(imm),
 [{
   uint8_t offset = Subtarget->getIORegisterOffset();
   uint64_t val = N->getZExtValue() - offset;
-  return val >= 0x0 && val < 0x40;
+  return val < 0x40;
 }], ioaddr_XFORM>;
 
 def lowioaddr8 : PatLeaf<(imm),
 [{
   uint8_t offset = Subtarget->getIORegisterOffset();
   uint64_t val = N->getZExtValue() - offset;
-  return val >= 0x0 && val < 0x20;
+  return val < 0x20;
 }], ioaddr_XFORM>;
 
 def ioaddr16 : PatLeaf<(imm),
 [{
   uint8_t offset = Subtarget->getIORegisterOffset();
   uint64_t val = N->getZExtValue() - offset;
-  return val >= 0x0 && val < 0x3f;
+  return val < 0x3f;
 }], ioaddr_XFORM>;
 
 def iobitpos8 : PatLeaf<(imm),