[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 06:25:04PM -0500, Rafael Avila de Espindola wrote:
> I so far have seem [] being used only one file and updated that one file 
> :-) I think Joerg mentioned it being common on old BSD code, so we might 
> have to support it.

[] is used in the default PIC prologue created by various compilers.

> Joerg, how common are assembly files with [] in them? If common, can you 
> code a patch making [] conditional on x86(-64) ELF?

Why not just make the function virtual and let platforms that can't
accept the default behavior overwrite it? I actually think that quite a
few platforms GNU as support don't distinguish between [] and () as the
patch implements.

Joerg