[lld] r336199 - [ELF] - Add a comment. NFC.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 5 11:49:04 PDT 2018


That is obvious and no need to explain, no?

On Tue, Jul 3, 2018 at 7:21 AM George Rimar via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> Author: grimar
> Date: Tue Jul  3 07:16:19 2018
> New Revision: 336199
>
> URL: http://llvm.org/viewvc/llvm-project?rev=336199&view=rev
> Log:
> [ELF] - Add a comment. NFC.
>
> Minor follow up for r336197
> "[ELF] - Add support for '||' and '&&' in linker scripts."
>
> Modified:
>     lld/trunk/ELF/ScriptLexer.cpp
>
> Modified: lld/trunk/ELF/ScriptLexer.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/ScriptLexer.cpp?rev=336199&r1=336198&r2=336199&view=diff
>
> ==============================================================================
> --- lld/trunk/ELF/ScriptLexer.cpp (original)
> +++ lld/trunk/ELF/ScriptLexer.cpp Tue Jul  3 07:16:19 2018
> @@ -116,6 +116,7 @@ void ScriptLexer::tokenize(MemoryBufferR
>      }
>
>      // ">foo" is parsed to ">" and "foo", but ">>" is parsed to ">>".
> +    // "|", "||", "&" and "&&" are different operators.
>      if (S.startswith("<<") || S.startswith("<=") || S.startswith(">>") ||
>          S.startswith(">=") || S.startswith("||") || S.startswith("&&")) {
>        Vec.push_back(S.substr(0, 2));
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180705/74d9ba9f/attachment.html>


More information about the llvm-commits mailing list