search for: d46701

...hange. > 2) [X86] Add variant scheduling classes for LEA instructions. > 3) [AArch64] Rewrite the predicates mentioned by this RFC. > > People that are interested in seeing how to implement "optional" patch 3 can > have a look at the review here: https://reviews.llvm.org/D46701 <https://reviews.llvm.org/D46701> > > Please let me know what you think. > > Thanks, > Andrea Fantastic writeup! It’s great to see so much progress on fundamental infrastructure. My time for LLVM code review is extremely limited. Can someone work with Andrea to get these pa...

...to review the llvm-mca change. 2) [X86] Add variant scheduling classes for LEA instructions. 3) [AArch64] Rewrite the predicates mentioned by this RFC. People that are interested in seeing how to implement "optional" patch 3 can have a look at the review here: https://reviews.llvm.org/D46701 Please let me know what you think. Thanks, Andrea -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20180510/f969d70d/attachment.html>

...hineInstrVariant" (both user defined functions). -- The bottom line is: the framework is extensible. As long as we tell the PredicateExpander how to lower/expand our new predicates, then we can implement different approaches. I hope this answers to the comments from https://reviews.llvm.org/D46701. Thanks, Andrea On Thu, May 10, 2018 at 10:24 PM, Renato Golin <renato.golin at linaro.org> wrote: > On 10 May 2018 at 21:58, Andrew Trick <atrick at apple.com> wrote: > > Fantastic writeup! It’s great to see so much progress on fundamental > > infrastructure. > &gt...

On 10 May 2018 at 21:58, Andrew Trick <atrick at apple.com> wrote: > Fantastic writeup! It’s great to see so much progress on fundamental > infrastructure. > > My time for LLVM code review is extremely limited. Can someone work with > Andrea to get these patches in? Hi Andrew, Same here, but this has been a long goal for me, too, so I'll do my best. -- cheers,