[PATCH] Adding altmacro support in integrated assembler. continue of D10591 diff #28219.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Fri Jun 26 04:23:58 PDT 2015


I can't apply this. Looks like the patch completely reindents parts of files:

-class MemoryBuffer;
-class MCAsmInfo;
+  class MemoryBuffer;
+  class MCAsmInfo;

-/// AsmLexer - Lexer class for assembly files.
-class AsmLexer : public MCAsmLexer {
-  const MCAsmInfo &MAI;
-
-  const char *CurPtr;
-  StringRef CurBuf;
-  bool isAtStartOfLine;


....

Probably the best way to fix this is just to run git-clang-format
before uploading the patch.


Cheers,
Rafael




On 24 June 2015 at 10:59, michael zuckerman <michael.zuckerman at intel.com> wrote:
> Hi grosbach, compnerd, tomatabacu, rafael,
>
> There was a problem with compere to base I did rebase.
> you can find more comments in link [[ URL |  http://reviews.llvm.org/D10591  ]]
> **`D10591`**
> The asm parser now support the altmacro and noaltmacro directives.
> This is flow the bug in link: https://llvm.org/bugs/show_bug.cgi?id=18918
> You can find information about altmacro in this link :https://sourceware.org/binutils/docs/as/Altmacro.html.
> In this part of the work we support the three type of string "..." , '...' ,<...>.
> You can now use '%' char to write `%expr’ to evaluate the expression expr and use the result as a string.
> In the future we will add the local directive and additional string support.
>
> http://reviews.llvm.org/D10692
>
> Files:
>   include/llvm/MC/MCParser/AsmLexer.h
>   include/llvm/MC/MCParser/MCAsmLexer.h
>   lib/MC/MCParser/AsmLexer.cpp
>   lib/MC/MCParser/AsmParser.cpp
>   test/MC/AsmParser/altMacroTest.s
>   test/MC/AsmParser/altMacroTestFail.s
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/




More information about the llvm-commits mailing list