[PATCH] ARM: Enforce decoding rules for VLD#

Amaury de la Vieuville Amaury.delaVieuville at arm.com
Thu Jun 6 01:34:28 PDT 2013


Hi Tim,

I updated the patch, VLDn and VSTn now share the same decoding checker methods.
What would you say about having decoders in the same fashion as DecodeVLD3DupInstruction() for these instructions instead of the huge DecodeV{ST,LD}Instruction()?

Thanks,
Amaury

From: Tim Northover [mailto:tnorthover at apple.com]
Sent: 06 June 2013 03:11
To: Amaury de la Vieuville
Cc: Tim Northover; llvm-commits
Subject: Re: [PATCH] ARM: Enforce decoding rules for VLD#

Hi Amaury,

Sorry for that, I attached it again.

Doesn't the actual logic there precisely duplicate the DecodeVSTn checks we recently added? I think I'd be happier if it was just in one place.

Cheers.

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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130606/d7a5c982/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vldn.diff
Type: application/octet-stream
Size: 26550 bytes
Desc: vldn.diff
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130606/d7a5c982/attachment.obj>


More information about the llvm-commits mailing list