No functional change (in supported use cases)
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM. Just like D3175, I'm not sure this patch doesn't introduce functional changes because you're changing the way our backend deals with Mips64/O32.
Comment Actions
Just like D3175, I'm not sure this patch doesn't introduce functional changes because you're changing the way our backend deals with Mips64/O32.
True, it would affect that case but that's supported yet and is likely to be broken since isMips64() has been used as a synonym for isN32()||isN64() in many places.