summaryrefslogtreecommitdiffstats
path: root/llvm/test/MC/Mips/mips-register-names-invalid.s
diff options
context:
space:
mode:
authorDaniel Sanders <daniel.sanders@imgtec.com>2014-04-01 10:35:28 +0000
committerDaniel Sanders <daniel.sanders@imgtec.com>2014-04-01 10:35:28 +0000
commitb50ccf8e26653781191491aab90c7379f29bd5e1 (patch)
tree3e29000817362dfc46f0653687c83535c7f10a59 /llvm/test/MC/Mips/mips-register-names-invalid.s
parent7fedc179d21f0f531d988021a4d5d5ad08c291ce (diff)
downloadbcm5719-llvm-b50ccf8e26653781191491aab90c7379f29bd5e1.tar.gz
bcm5719-llvm-b50ccf8e26653781191491aab90c7379f29bd5e1.zip
[mips] Rewrite MipsAsmParser and MipsOperand.
Summary: Highlights: - Registers are resolved much later (by the render method). Prior to that point, GPR32's/GPR64's are GPR's regardless of register size. Similarly FGR32's/FGR64's/AFGR64's are FGR's regardless of register size or FR mode. Numeric registers can be anything. - All registers are parsed the same way everywhere (even when handling symbol aliasing) - One consequence is that all registers can be specified numerically almost anywhere (e.g. $fccX, $wX). The exception is symbol aliasing but that can be easily resolved. - Removes the need for the hasConsumedDollar hack - Parenthesis and Bracket suffixes are handled generically - Micromips instructions are parsed directly instead of going through the standard encodings first. - rdhwr accepts all 32 registers, and the following instructions that previously xfailed now work: ddiv, ddivu, div, divu, cvt.l.[ds], se[bh], wsbh, floor.w.[ds], c.ngl.d, c.sf.s, dsbh, dshd, madd.s, msub.s, nmadd.s, nmsub.s, swxc1 - Diagnostics involving registers point at the correct character (the $) - There's only one kind of immediate in MipsOperand. LSA immediates are handled by the predicate and renderer. Lowlights: - Hardcoded '$zero' in the div patterns is handled with a hack. MipsOperand::isReg() will return true for a k_RegisterIndex token with Index == 0 and getReg() will return ZERO for this case. Note that it doesn't return ZERO_64 on isGP64() targets. - I haven't cleaned up all of the now-unused functions. Some more of the generic parser could be removed too (integers and relocs for example). - insve.df needed a custom decoder to handle the implicit fourth operand that was needed to make it parse correctly. The difficulty was that the matcher expected a Token<'0'> but gets an Imm<0>. Adding an implicit zero solved this. Reviewers: matheusalmeida, vmedic Reviewed By: matheusalmeida Differential Revision: http://llvm-reviews.chandlerc.com/D3222 llvm-svn: 205292
Diffstat (limited to 'llvm/test/MC/Mips/mips-register-names-invalid.s')
-rw-r--r--llvm/test/MC/Mips/mips-register-names-invalid.s2
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/test/MC/Mips/mips-register-names-invalid.s b/llvm/test/MC/Mips/mips-register-names-invalid.s
index df1054fed42..e6f8416a41e 100644
--- a/llvm/test/MC/Mips/mips-register-names-invalid.s
+++ b/llvm/test/MC/Mips/mips-register-names-invalid.s
@@ -4,5 +4,5 @@
# $32 used to trigger an assertion instead of the usual error message due to
# an off-by-one bug.
-# CHECK: :[[@LINE+1]]:18: error: invalid operand for instruction
+# CHECK: :[[@LINE+1]]:17: error: invalid operand for instruction
add $32, $0, $0
OpenPOWER on IntegriCloud