[PATCH] D19714: [mips][ias] Split expandMemInst between MipsAsmParser and MipsTargetStreamer. Almost NFC.

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 29 01:34:01 PDT 2016


dsanders created this revision.
dsanders added a reviewer: sdardis.
dsanders added a subscriber: llvm-commits.
dsanders added a dependency: D19712: [mips][ias] Moved most instruction emission helpers to MipsTargetStreamer. NFC..
Herald added subscribers: sdardis, dsanders.

The portion in MipsAsmParser is responsible for figuring out which expansion to
use, while the portion in MipsTargetStreamer is responsible for emitting it.

This allows us to remove the call to isIntegratedAssemblerRequired() which is
currently ensuring the effect of .cprestore only occurs when writing objects.

The small functional change is that the memory offsets are now correctly
printed as signed values.

Depends on D19712.

http://reviews.llvm.org/D19714

Files:
  lib/Target/Mips/AsmParser/MipsAsmParser.cpp
  lib/Target/Mips/MCTargetDesc/MipsTargetStreamer.cpp
  lib/Target/Mips/MipsTargetStreamer.h
  test/MC/Mips/micromips-expansions.s
  test/MC/Mips/mips-expansions.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19714.55545.patch
Type: text/x-patch
Size: 15814 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160429/23d42675/attachment.bin>


More information about the llvm-commits mailing list