[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?
More information about the llvm-commits