[PATCH] D41183: [mips] Fix the target specific instruction verifier

Simon Dardis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 13 06:40:30 PST 2017


sdardis created this revision.
sdardis added a reviewer: atanasyan.
Herald added a subscriber: arichardson.

Fix an off by one error in the bounds checking for 'dinsu' and update
the ranges in the test comments so that they are accurate.


Repository:
  rL LLVM

https://reviews.llvm.org/D41183

Files:
  lib/Target/Mips/MipsInstrInfo.cpp
  test/CodeGen/Mips/fcopysign-f32-f64.ll
  test/CodeGen/Mips/fcopysign.ll
  test/CodeGen/Mips/instverify/dext-pos.mir
  test/CodeGen/Mips/instverify/dext-size.mir
  test/CodeGen/Mips/instverify/dextm-pos-size.mir
  test/CodeGen/Mips/instverify/dextm-pos.mir
  test/CodeGen/Mips/instverify/dextm-size.mir
  test/CodeGen/Mips/instverify/dextu-pos-size.mir
  test/CodeGen/Mips/instverify/dextu-pos.mir
  test/CodeGen/Mips/instverify/dextu-size-vaild.mir
  test/CodeGen/Mips/instverify/dextu-size.mir
  test/CodeGen/Mips/instverify/dins-pos-size.mir
  test/CodeGen/Mips/instverify/dins-pos.mir
  test/CodeGen/Mips/instverify/dins-size.mir
  test/CodeGen/Mips/instverify/dinsm-pos-size.mir
  test/CodeGen/Mips/instverify/dinsm-pos.mir
  test/CodeGen/Mips/instverify/dinsm-size.mir
  test/CodeGen/Mips/instverify/dinsu-pos-size.mir
  test/CodeGen/Mips/instverify/dinsu-pos.mir
  test/CodeGen/Mips/instverify/dinsu-size.mir
  test/CodeGen/Mips/instverify/ext-pos-size.mir
  test/CodeGen/Mips/instverify/ext-pos.mir
  test/CodeGen/Mips/instverify/ext-size.mir
  test/CodeGen/Mips/instverify/ins-pos-size.mir
  test/CodeGen/Mips/instverify/ins-pos.mir
  test/CodeGen/Mips/instverify/ins-size.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D41183.126756.patch
Type: text/x-patch
Size: 17405 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171213/fdc6f60a/attachment.bin>


More information about the llvm-commits mailing list