[llvm] r310243 - [ARM] Fix assembly and disassembly for VMRS/VMSR

Andre Simoes Dias Vieira via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 25 02:06:02 PDT 2017


Hello,

I noticed the ticket still has the "ready to land" status on Phabricator
from the previous version. I was expecting its status to reset after I
had re-opened it.

Could you just confirm you are happy with its current status and it
passes your tests Tim?

Cheers,
Andre

On 21/08/17 12:05, Andre Simoes Dias Vieira wrote:
> Hi Tim,
>
> I have updated https://reviews.llvm.org/D36306 to reflect your comments.
>
> Sorry for the delay.
>
> Cheers,
> Andre
>
> On 08/08/17 18:19, Tim Northover wrote:
>> On 8 August 2017 at 09:45, Andre Simoes Dias Vieira
>> <Andre.SimoesDiasVieira at arm.com> wrote:
>>> I am about to go on holidays though, I will have another go at it when
>>> I'm back in a week.
>>
>> OK, since the patch is essentially improving diagnostics I think it's
>> probably better to revert for now so I've done that in r310384.
>>
>> Tim.
>>
>

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


More information about the llvm-commits mailing list