[PATCH] [AArch64] Refactor AArch64NamedImmMapper to become dependent on subtarget features.

Eric Christopher echristo at gmail.com
Tue Apr 21 08:32:49 PDT 2015


Please just make the changes I've requested in a follow-up commit. I don't
see the need to reply on another commit.

On Tue, Apr 21, 2015, 3:16 AM Vladimir Sukharev <vladimir.sukharev at arm.com>
wrote:

> Eric, thank you for your feedback
> in fact, I have already committed that as http://reviews.llvm.org/rL235089,
> with slight re-work, addressing last jmolloy's comment
> I suspect a line of text after "Differential Revision:" line has prevented
> post-commit script to process auto-closing this DiffRevision properly. And
> I don't know at the time how should I fix that: close Revision and put
> links to commit and back. (Is it a way to do that at all?)
> Anyway, should I address your comments as a separate follow-up patch? If
> so, would you please comment directly to http://reviews.llvm.org/rL235089,
> because things are changed a bit.
>
>
> REPOSITORY
>   rL LLVM
>
> http://reviews.llvm.org/D8496
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150421/5627b4b7/attachment.html>


More information about the llvm-commits mailing list