[PATCH] D19873: [mips][micromips] Make getPointerRegClass() result depend on the instruction.

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Tue May 3 09:31:11 PDT 2016


dsanders created this revision.
dsanders added reviewers: hvarga, zbuljan, zoran.jovanovic, sdardis.
dsanders added a subscriber: llvm-commits.
Herald added subscribers: sdardis, dsanders.

Previously, it returned the GPR16MMRegClass for all instructions which was
incorrect for instructions like lwsp/lwgp and unnecesarily restricted the
permitted registers for instructions like lw32.

This fixes quite a few of the -verify-machineinstrs errors reported in PR27458.
I've only added -verify-machineinstrs to one test in this change since I
understand there is a plan to enable the verifier by default.

http://reviews.llvm.org/D19873

Files:
  lib/Target/Mips/AsmParser/MipsAsmParser.cpp
  lib/Target/Mips/MicroMipsInstrInfo.td
  lib/Target/Mips/MipsRegisterInfo.cpp
  lib/Target/Mips/MipsRegisterInfo.h
  lib/Target/Mips/MipsRegisterInfo.td
  test/CodeGen/Mips/micromips-addiu.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19873.56016.patch
Type: text/x-patch
Size: 6278 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160503/7eebf639/attachment.bin>


More information about the llvm-commits mailing list