[PATCH] D30941: Better testing of schedule model instruction latencies/throughputs

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 21 06:36:50 PDT 2017


RKSimon added a comment.

In https://reviews.llvm.org/D30941#703613, @hfinkel wrote:

> In https://reviews.llvm.org/D30941#703612, @avt77 wrote:
>
> > hfinkel, yes I mean I'll do it in the next version of this patch soon.
> >  rksimon, do you mean we should rename the compiler option like "-print-schedule"?
>
>
> It is not clear to me why you won't just always do this when in verbose-asm mode. Thoughts on not having a separate option at all?


@hfinkel It looks like putting this under -verbose-asm instead of -print-schedule will cause a lot (643!) of test failures that will need fixing - is that something we should be looking to do?


https://reviews.llvm.org/D30941





More information about the llvm-commits mailing list