[PATCH] D11221: Fix crash when using __attribute__((section (".bss.name"))) incorrectly

[Jakub Kuderski]

kuhar added a comment.

In http://reviews.llvm.org/D11221#207708, @joerg wrote:

> Why should this be in the frontend? Arguable, writing assembler with this is not wrong and if anything, the diagnostic should really be in the MC layer. I don't think precision of diagnostic is really that high a priority here, which would be the normal concern with errors from the MC layer. I don't think this is different from declaring multiple symbols of the same name etc.


I'm a Clang newbie and I'm not really sure which approach is the best.
Richard, what do you think about this? I would really appreciate your opinion and some help.


Repository:
  rL LLVM

http://reviews.llvm.org/D11221