On 7/23/24 19:48, Kito Cheng wrote:
I incline do not add skip_zacas stuffs (although skip_zabha is already
there but that's fine), because that's different situation compare to
the zaamo/zalrsc, zaamo/zalrsc should automatically append if a
extension is available, which is new behavior and new extensions.
But zacas is only added when users explicitly add that in -march
string unlike zaamo/zalrsc, so I am not sure if we need to check the
binutils support and drop that if unsupported,
My biggest concern is : should we do so for every new extension?
I think we didn't do that so far, so we should
I included it since binutils only recently added support but you make a
good point.
Failing explicitly for new extensions on outdated binutils makes sense
to me.
If there aren't any objections I'll remove it from v3 and send a
separate patch to
remove zabha so it's clear we only handle breaking changes to existing
extensions.
