[PATCH] D21728: [AARCH64] Add Broadcom Vulcan scheduling model

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 28 15:17:57 PDT 2016


This never made it to the list as a patch afaict. Can you resend the patch
with llvm-commits as a subscriber first?

On Tue, Jun 28, 2016, 2:51 PM Renato Golin <renato.golin at linaro.org> wrote:

> rengolin accepted this revision.
> rengolin added a comment.
> This revision is now accepted and ready to land.
>
> Hi Pankaj,
>
> I don't know much about the Vulcan model, so I'll trust you on the
> numbers. :)
>
> But the code looks good to me with the extra comment removed.
>
> cheers,
> --renato
>
>
> ================
> Comment at: lib/Target/AArch64/AArch64SchedVulcan.td:445
> @@ +444,3 @@
> +
> +// Load vector reg, literal
> +// Load vector reg, unscaled immed
> ----------------
> I don't think you need this section here... you can easily say above that
> it's valid for FP, too.
>
>
> http://reviews.llvm.org/D21728
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160628/92b6ec68/attachment.html>


More information about the llvm-commits mailing list