On 10/19/2017 08:11 AM, Dong Jia Shi wrote: >> + if (!sch || !css_subch_visible(sch)) { >> + setcc(cpu, 3); > ?: > s/3/IOINST_CC_NOT_OPERATIONAL/ > > This also applies to other alike cases. >
I do not know. My first reaction was that I'm against because were IOInstEnding strongly typed, it would not work as set cc takes an int and works not just for IO instructions. OTOH the same goes for setcc(cpu, css_do_xsch(sch)) so that ain't really an argument, and it would improve grepability. So I think I'm in favor now. Halil