[llvm-commits] [llvm] r125595 - in /llvm/trunk: lib/MC/MCParser/AsmParser.cpp test/MC/AsmParser/exprs.s test/MC/AsmParser/paren.s

[Joerg Sonnenberger]

On Wed, Feb 16, 2011 at 09:47:22AM -0800, Jim Grosbach wrote:
> What about other targets? ARM, MIPS, etc.. shouldn't accept
> bracket-as-parens expressions unless that's part of their documented
> syntax.

Can you name a specific platform for which GAS doesn't accept it?

> For example, the following is not legal ARM assembly and should give a
> syntax error on the '[' token.
> add r0, r1, #[_foo - _bar]
> 
> With these changes, that's not happening anymore.

_foo:
_bar:
	add r0, r1, #[_foo - _bar]

is accepted by GAS and gives the same result as the expression with ().

Joerg