As discussed in #127536 (comment), the naming style in insFlags is not consistent. The quick fix could be having all the flag names converge to INS_FLAGS_* to match with the base flag - INS_FLAGS_None.
For the long term, the definition can switch to enum class insFlags, so the flags could be accessed by insFlags::<Flag name>. It then will require some refactoring in instrsxarch.h and emitter.
As discussed in #127536 (comment), the naming style in
insFlagsis not consistent. The quick fix could be having all the flag names converge toINS_FLAGS_*to match with the base flag -INS_FLAGS_None.For the long term, the definition can switch to
enum class insFlags, so the flags could be accessed byinsFlags::<Flag name>. It then will require some refactoring ininstrsxarch.hand emitter.