[PATCH] D64252: [Mips] Instruction `sc` now accepts symbol as an argument

Mirko Brkusanin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 5 08:52:43 PDT 2019


mbrkusanin created this revision.
mbrkusanin added reviewers: atanasyan, mstojanovic, petarj, sdardis.
mbrkusanin added a project: LLVM.
Herald added a subscriber: arichardson.

Function MipsAsmParser::expandMemInst() did not properly handle instruction `sc`
with a symbol as an argument because first argument would be counted twice. 
By removing unnecessary operand from instruction definition symbols can be 
handled without any changes to MipsAsmParser.


https://reviews.llvm.org/D64252

Files:
  lib/Target/Mips/Disassembler/MipsDisassembler.cpp
  lib/Target/Mips/MCTargetDesc/MipsNaClELFStreamer.cpp
  lib/Target/Mips/MicroMips32r6InstrInfo.td
  lib/Target/Mips/MicroMipsInstrInfo.td
  lib/Target/Mips/Mips32r6InstrInfo.td
  lib/Target/Mips/MipsEVAInstrInfo.td
  lib/Target/Mips/MipsExpandPseudo.cpp
  lib/Target/Mips/MipsInstrInfo.td
  test/CodeGen/Mips/micromips-eva.mir
  test/MC/Mips/sym-sc.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D64252.208186.patch
Type: text/x-patch
Size: 12039 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190705/12e96cd3/attachment.bin>


More information about the llvm-commits mailing list