[patch] Add support for .exitm in the integrated assembler

[Rafael Espíndola]

LGTM

On 24 July 2014 12:51, Nico Weber <thakis at chromium.org> wrote:
> On Thu, Jul 24, 2014 at 9:32 AM, Saleem Abdulrasool <compnerd at compnerd.org>
> wrote:
>>
>> On Wed, Jul 23, 2014 at 10:59 PM, Nico Weber <thakis at chromium.org> wrote:
>>>
>>> Hi,
>>>
>>> the attached patch adds support for .exitm as documented on
>>> https://sourceware.org/binutils/docs/as/Exitm.html#Exitm , PR20426.
>>> Everything from the current macro expansion after .exitm is ignored; this
>>> matches gas's behavior.
>>>
>>> (It also removes the unused TheMacro member from MacroInstantiation.)
>>
>>
>> Can you split this up into two patches.  Removing the unused member
>> variable is fine, but Id rather that be a separate change.
>
>
> Sure, did that part in .213874.
>
>>
>> Unnecessary braces in parseDirectiveExitMacro around the
>> isInsideMacroInstantiation().
>>
>> It feels like the tests for the .exitm could be simplified further.
>>
>> .exitm is encountered in a normal macro expansion
>> .exitm is in a true branch
>> .exitm is in a false branch
>> .exitm is in a false branch that is encountered prior to the true branch
>
>
> All done. I kept a test for a macro expanded inside a conditional, for
> coverage of popping if states only up to where they were when macro
> expansion began.
>
>>
>>
>>>
>>> Please take a look!
>>>
>>> Nico
>>>
>>> _______________________________________________
>>> llvm-commits mailing list
>>> llvm-commits at cs.uiuc.edu
>>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>>>
>>
>>
>>
>> --
>> Saleem Abdulrasool
>> compnerd (at) compnerd (dot) org
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>