[PATCH] D49598: [SystemZ] Use tablegen loops in SchedModels

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 23 11:46:10 PDT 2018


uweigand added a comment.

I believe in this case it would be preferable to include the full range of latency values.  We should try to keep the source as concise and straightforward as possible; if this results in some units being defined that aren't used, that doesn't seem to be a drawback to me.   Also, for the common code parts, even if no current instruction uses any particular value, there may always be additional instructions in future processors.


https://reviews.llvm.org/D49598





More information about the llvm-commits mailing list