<div>Hello Kevin,</div><div><br></div><div>I thinking about this, but there are some problems:</div><div><br></div><div>1) Differrent interpretation of $[:number:] in macro body:</div><div>source:</div><div>.macro test par1</div>
<div>  movl $0, %eax</div><div>.endm</div><div><br></div><div>test %ebx</div><div><br></div><div>translated to:</div><div>original llvm => movl %ebx, %eax</div>

<div>with patch => movl $0, %ebx</div><div><br></div><div>2) Different parsing of space in macro parameters:</div><div>source:</div><div><br></div><div>test2 a + b,c</div><div><br></div><div>parsed as:</div><div>original llvm => macro test2 with two arguments 'a+b','c'</div>
<div>with patch => macro test2 with three arguments 'a','+','b,c'</div>

<div><br></div><div>For compability this problems requiring some compiler switch flag. Can you give me description/example how it's can be done?</div><br><div>Thanks for response.</div><div><br></div><div class="gmail_quote">
2012/2/9 Kevin Enderby <span dir="ltr"><<a href="mailto:enderby@apple.com" target="_blank">enderby@apple.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div style="word-wrap:break-word">Hello Vladimir,<div><br></div><div>While this is great for platforms that use the current gas style assembler macros your patch also changes the existing support for darwin that does not use these style of assembler macros.  Could you please update your patch that leaves the assembler macros unchanged for darwin.  Then we can review the updated patch.</div>


<div><br></div><div>Thanks,</div><div>Kev</div><div><br><div><div><div>On Feb 5, 2012, at 1:43 PM, Vladimir Sorokin wrote:</div><br></div><blockquote type="cite"><div><div></div><div><div>Hello llvm users!</div>
<div><br></div>I make patch of asm parser for make it to be closer to gas:<div style="background-image:initial!important;background-color:transparent!important;padding-top:0px!important;padding-right:0px!important;padding-bottom:0px!important;padding-left:0px!important;margin-top:0px!important;margin-right:0px!important;margin-bottom:0px!important;margin-left:0px!important;text-align:left!important;background-repeat:initial initial!important">



<div><div></div><div></div></div><img src="http://www.google.com/uds/css/small-logo.png"></div>
<div><br></div><div>1) treat space as comma of parameters</div><div>2) support "req" & "vararg" atttributes</div><div>3) macro calls with with explicit named arguments</div><div><br></div><div>Path attached. Can anybody review it?</div>



</div></div><div><span><asm-macro-ext.patch></span>_______________________________________________<br>LLVM Developers mailing list<br><a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>


<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br></div></blockquote></div><br></div></div></blockquote></div><br>
<div style="border-color:none!important;background:transparent!important;padding:0!important;margin:0!important;overflow:visible!important;text-align:left!important"><div><div>
</div><div></div></div><img src="http://www.google.com/uds/css/small-logo.png"></div>