On Wed, 3 Mar 2021, Jeff Law wrote: > > gcc/testsuite/ChangeLog: > > * gcc.target/mips/compact-branches-1.c: add isa_rev>=6. > > * gcc.target/mips/mips.exp: don't add -mipsXXr6 option for > > -mcompact-branches=always. It is usable for pre-R6 now. > > * gcc.target/mips/compact-branches-8.c: New test. > > * gcc.target/mips/compact-branches-9.c: New test. > So I think Maciej's comment was that you simply shouldn't be using > -mcompact-branches=always at mips32r2 (or anything pre-r6) together. > > I think what you're trying to do here is set up a scenario where you're > defaulting to mips32r6 and compact-branches, but not error if something > specifies -mcpu=mips32r2 or something similar, right?
Actually what I had in mind was relaxing the strictness of the `always' case. I have replied in a more elaborate manner in the original thread where I raised my concerns. I have not looked through the code of any newer proposals. Maciej
