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

[Jim Grosbach]

Hi Joerg,

I'm sorry that we've been unable to resolve this via discussion. Perhaps we simply have differing enough philosophical approaches that we simply weren't ever going to reach consensus.

Be that as it may, I do feel strongly that it is inappropriate for this change to go in as it stands. It introduces ambiguities into the ARM syntax, as I previously documented, for an X86 extenstion. I have reverted the patch in r126336. If you would like to resubmit with the feature being conditional on X86/ELF, please do.

If you feel this is inappropriate or would otherwise like to escalate the issue, please contact the LLVM backend maintainer, Evan Cheng. I've CCed him on this message.

Regards,
  Jim