[PATCH] D11221: Fix crash when using __attribute__((section (".bss.name"))) incorrectly
Jakub Kuderski
jakub.kuderski at arm.com
Mon Jul 20 02:56:22 PDT 2015
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
More information about the cfe-commits
mailing list