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

[Jim Grosbach]

On Feb 16, 2011, at 10:12 AM, Joerg Sonnenberger wrote:

> On Wed, Feb 16, 2011 at 09:47:22AM -0800, Jim Grosbach wrote:
>>> AT&T syntax doesn't care about [], I think. Intel syntax is messy as best,
>>> but if it could only need a virtual override.
>> 
>> What about other targets? ARM, MIPS, etc.. shouldn't accept
>> bracket-as-parens expressions unless that's part of their documented syntax.
> 
> Do we really want to repeat the mistakes of GNU as? Having a slightly
> different assembler syntax on every platforms just sucks. So unless
> there is a very good reason, I would strongly discourage having such
> diversity.

That's not a choice that's up to us. We're not deciding what the syntax is. We're supporting existing (hopefully documented) syntax.

-Jim