Hi James, Thanks for reviewing the patch and comments.
>> I wonder whether the current modeling of:
>> (define_insn_reservation "thunderx2t99_asimd_load4_elts" 6
>> Actually benefits the schedule in a meaningful way, or if it just increases
Done. Removed the scheduler modeling for thunderx2t99_asimd_load*_mult and
thunderx2t99_asimd_load*_elts for ld3/ld4 and st3/st4 which are rarely used.
The automaton size has come down drastically without that and hopefully
should be okay.
============================================================
Automaton `thunderx2t99'
184 NDFA states, 838 NDFA arcs
184 DFA states, 838 DFA arcs
184 minimal DFA states, 838 minimal DFA arcs
360 all insns 8 insn equivalence classes
0 locked states
1016 transition comb vector els, 1472 trans table els: use simple vect
1472 min delay table els, compression factor 4
Automaton `thunderx2t99_advsimd'
453 NDFA states, 1966 NDFA arcs
453 DFA states, 1966 DFA arcs
351 minimal DFA states, 1562 minimal DFA arcs
360 all insns 7 insn equivalence classes
0 locked states
1901 transition comb vector els, 2457 trans table els: use simple vect
2457 min delay table els, compression factor 2
Automaton `thunderx2t99_ldst'
41 NDFA states, 163 NDFA arcs
41 DFA states, 163 DFA arcs
14 minimal DFA states, 78 minimal DFA arcs
360 all insns 8 insn equivalence classes
0 locked states
83 transition comb vector els, 112 trans table els: use simple vect
112 min delay table els, compression factor 4
Automaton `thunderx2t99_mult'
2 NDFA states, 5 NDFA arcs
2 DFA states, 5 DFA arcs
2 minimal DFA states, 5 minimal DFA arcs
360 all insns 3 insn equivalence classes
0 locked states
6 transition comb vector els, 6 trans table els: use simple vect
6 min delay table els, compression factor 8
============================================================
>> You'll want to update this to use your new scheduling model :-).
Done. I had overlooked it :-).
>> you should be changing vulcan to use the new thunderx2t99 model.
Done. Using the new thunderx2t99 model.
Please review the modified patch and let us know your comments on the same.
Thanks,
Naveen
thunderx2t99-scheduler.patch
Description: thunderx2t99-scheduler.patch
