]> granicus.if.org Git - llvm/commit
[mips] Add new error message and improve testing for parsing the .module directive.
authorToma Tabacu <toma.tabacu@imgtec.com>
Fri, 23 Jan 2015 10:40:19 +0000 (10:40 +0000)
committerToma Tabacu <toma.tabacu@imgtec.com>
Fri, 23 Jan 2015 10:40:19 +0000 (10:40 +0000)
commitd6fbb91ea28567c86d27ab27d09a4d6e1bd124dc
tree874cb054d3d038241482358d883955f6fe3a85ac
parent245caec9b38878ebb5b4510b5f477ed063cd9653
[mips] Add new error message and improve testing for parsing the .module directive.

Summary:
We used to silently ignore any empty .module's and we used to give an error saying that we found
an "unexpected token at start of statement" when the value of the option wasn't an identifier (e.g. if it was a number).

We now give an error saying that we "expected .module option identifier" in both of those cases.

I also fixed the other tests in mips-abi-bad.s, which all seemed to be broken.

Reviewers: dsanders

Reviewed By: dsanders

Subscribers: llvm-commits

Differential Revision: http://reviews.llvm.org/D7095

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@226905 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Target/Mips/AsmParser/MipsAsmParser.cpp
test/MC/Mips/mips-abi-bad.s