The -mcpu=mips16 option caused the Integrated Assembler to crash because
it couldn't figure out the architecture revision number to write to the
.MIPS.abiflags section. This CPU definition has been removed because, like
microMIPS, MIPS16 is an ASE to a base architecture.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Could you update MipsABIInfo::computeTargetABI() too?
I hadn't noticed that one. I'll fix that in the commit
Comment Actions
I haven't changed anything on that side of things, clang was already using it as an ASE. The
-mips16/-mno-mips16 options are still using -target-feature to pass the mips16 feature bit
to the backend and the mixed mode support is still being done via function attributes. I
also checked our internal buildbot was using -mips16 rather than -mcpu=mips16 before
committing.