<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Dec 5, 2013 at 11:46 AM, David Peixotto <span dir="ltr"><<a href="mailto:dpeixott@codeaurora.org" target="_blank">dpeixott@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Please help to review this patch.<br>
<br>
The integrated assembler fails to properly lex arm comments when<br>
they are adjacent to an identifier in the input stream. The reason<br>
is that the arm comment symbol '@' is also used as symbol variant in<br>
other assembly languages so when lexing an identifier it allows the<br>
'@' symbol as part of the identifier.<br></blockquote><div><br></div><div>Am I missing something or does this actually break handling for PLT and GOT relative symbols?  (e.g. sym@GOT).  This would prevent usage of the IAS with PIC code for ELF targets then I suspect.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Example:<br>
  $ cat comment.s<br>
  foo:<br>
    add r0, r0@got to parse this as a comment<br>
<br>
  $ llvm-mc -triple armv7 comment.s<br>
  comment.s:4:18: error: unexpected token in argument list<br>
    add r0, r0@got to parse this as a comment<br>
                   ^<br>
This should be parsed as correctly as `add r0, r0`.<br>
<br>
This commit modifies the assembly lexer to not include the '@' symbol<br>
in identifiers when lexing for targets that use '@' for comments.<br>
<br>
-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted<br>
by The Linux Foundation<br>
<br>
<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br>Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org
</div></div>