[PATCH] D39117: [mips] Match 'ins' and its' variants with C++ code

Simon Dardis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 20 03:13:50 PDT 2017


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

Change the ISel matching of 'ins', 'dins[mu]' from tablegen code to
C++ code. This resolves an issue where ISel would select 'dins' instead
of 'dinsm' when the instructions size and position were individually in
range but their sum was out of range according to the ISA specification.


Repository:
  rL LLVM

https://reviews.llvm.org/D39117

Files:
  lib/Target/Mips/MicroMips64r6InstrInfo.td
  lib/Target/Mips/MicroMipsInstrInfo.td
  lib/Target/Mips/Mips64InstrInfo.td
  lib/Target/Mips/MipsInstrInfo.td
  lib/Target/Mips/MipsSEISelDAGToDAG.cpp
  test/CodeGen/Mips/dins.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39117.119642.patch
Type: text/x-patch
Size: 7151 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171020/536730fe/attachment.bin>


More information about the llvm-commits mailing list