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

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa added a comment.

Thanks for helping me out with TableGen coding, Javed!

> But you have LSU, LSU2, LSU3, ... . The first one would have to change to LSU1

I have sofar tried to avoid LSU1, FXa1, so I did

  foreach Num = ["", "2", "3", "4", "5"]

, which seems to work. I would think this is ok, or? Only drawback here is that now there are in the subtarget files a separate definition for the "implicit 1\
" in some places. Not sure which is better...

Compared to trunk, there are in the tablegen file some extra variants, like FXa5 in z14 file, but that did not seem to matter - it seems that the produced cla\
ng binary is identical to that of trunk.

Any suggestions most welcome!


https://reviews.llvm.org/D49598