<div dir="ltr">Hi,<div><br></div><div>Tom, I relent and am happy with it available universally if Jim is.</div><div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><div class="gmail_extra"><br><div class="gmail_quote">On 29 September 2014 17:37, Jim Grosbach <span dir="ltr"><<a href="mailto:grosbach@apple.com" target="_blank">grosbach@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Tom,<br>
<br>
I have a slight preference for allowing the alternative universally, but don’t object to putting it behind a flag. Since this is behavior that will be relevant to end-users, not just us compiler folks, however, the flag should be at the clang driver level, not just a backend flag as this patch implements.<br>
<br>
-Jim<br>
<div><div class="h5"><br>
<br>
> On Sep 29, 2014, at 3:32 AM, Tom Coxon <<a href="mailto:tom.coxon@arm.com">tom.coxon@arm.com</a>> wrote:<br>
><br>
> Hi Jim,<br>
><br>
> Could you let me know which patch would be preferred? The new one is attached. The previous one you approved (but was shot down by James Molloy) didn't include the command line option.<br>
><br>
> Thanks,<br>
> Tom<br>
><br>
> -----Original Message-----<br>
> From: Tom Coxon [mailto:<a href="mailto:tom.coxon@arm.com">tom.coxon@arm.com</a>]<br>
> Sent: 19 September 2014 17:02<br>
> To: 'Jim Grosbach'<br>
> Cc: llvm-commits; '<a href="mailto:james@jamesmolloy.co.uk">james@jamesmolloy.co.uk</a>'<br>
> Subject: RE: [PATCH][AArch64] Enable access to the whole MRS/MSR system reg encoding space<br>
><br>
> Hi Jim,<br>
><br>
> Thanks for approving the patch. In case you missed it, James Molloy raised a concern:<br>
><br>
>> However, the change makes the production compiler less strict, and I<br>
>> don't think that's right. This change should be enabled behind a flag<br>
>> so that a user explicitly has to opt-in to it.<br>
><br>
> I've attached an alternative patch that enables it only when -aarch64-enable-undefined-sysregs is used. Could you let me know your preference?<br>
><br>
> Thanks,<br>
> Tom<br>
><br>
><br>
> -----Original Message-----<br>
> From: Jim Grosbach [mailto:<a href="mailto:grosbach@apple.com">grosbach@apple.com</a>]<br>
> Sent: 18 September 2014 17:55<br>
> To: Tom Coxon<br>
> Cc: llvm-commits<br>
> Subject: Re: [PATCH][AArch64] Enable access to the whole MRS/MSR system reg encoding space<br>
><br>
</div></div><div><div class="h5">> Huh. I thought we already did that for exactly the reasons you state. In any case, sounds reasonable to me. Patch looks fine, please commit.<br>
><br>
> -Jim<br>
><br>
>> On Sep 18, 2014, at 6:57 AM, Tom Coxon <<a href="mailto:tom.coxon@arm.com">tom.coxon@arm.com</a>> wrote:<br>
>><br>
>> Hi all,<br>
>><br>
>> The attached patch enables access to the whole space of ARMv8-A system registers through the S<op0>_<op1>_<Cn>_<Cm>_<op2> syntax.<br>
>><br>
>> This is useful to anyone needing to write assembly code supporting new system registers before the assembler has learned the official names for them.<br>
>><br>
>> Although this is a real use case, it is niche and most users should still use the official system registers names. In order to avoid encouraging general use of this syntax, ARM is not planning to document this in the ARMARM.<br>
>><br>
>> Regards,<br>
>> Tom<br>
>><br>
>> Files changed:<br>
>> lib/Target/AArch64/AArch64InstrFormats.td          |   12 +++----<br>
>> .../AArch64/Disassembler/AArch64Disassembler.cpp   |   24 +++----------<br>
>> .../AArch64/InstPrinter/AArch64InstPrinter.cpp     |   12 +++----<br>
>> lib/Target/AArch64/Utils/AArch64BaseInfo.cpp       |   36 +++++++-------------<br>
>> lib/Target/AArch64/Utils/AArch64BaseInfo.h         |    2 +-<br>
>> test/MC/AArch64/arm64-system-encoding.s            |    4 +++<br>
>> test/MC/AArch64/basic-a64-diagnostics.s            |   16 ++++-----<br>
>> test/MC/AArch64/basic-a64-instructions.s           |    8 +++--<br>
>> 8 files changed, 44 insertions(+), 70 deletions(-)<br>
>> <0001-AArch64-Allow-access-to-all-system-registers-with-MR.patch><br>
><br>
><br>
</div></div>> <0001-AArch64-Allow-access-to-all-system-registers-with-MR.patch><br>
<br>
</blockquote></div><br></div>