Page MenuHomeFreeBSD

Replace two-operand instruction aliases that clang does not understand with the three-operand instructions.
AbandonedPublic

Authored by pfg on Feb 15 2019, 4:13 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 20 2023, 1:26 AM
Unknown Object (File)
Nov 9 2023, 10:48 PM
Unknown Object (File)
Oct 8 2023, 9:45 PM
Unknown Object (File)
Sep 20 2023, 8:15 AM
Unknown Object (File)
Aug 24 2023, 8:59 AM
Unknown Object (File)
May 14 2023, 6:26 PM
Unknown Object (File)
Apr 25 2023, 6:46 PM
Subscribers

Details

Reviewers
jhb
Summary

No binary change with gas.
Obtained from: OpenBSD (CVS Rev. 1.9)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22515
Build 21662: arc lint + arc unit

Event Timeline

Note that I have no idea of MIPS asm, just noticed the commit on OpenBSD MIPS64 .. feel free to commit if it works for you.

I haven't needed this when building mips64 on clang 6. clang 7 can't build mips due to a different regression (but not in this file).

In D19203#410890, @jhb wrote:

I haven't needed this when building mips64 on clang 6. clang 7 can't build mips due to a different regression (but not in this file).

IIRC, OpenBSD needed this only for older versions of clang that didn't really support mips assembler completely (can't recall if those were the in-tree ones, or the snapshots by the folks doing the mips port). I helped Ed with a number of these a while ago. My sense is that we shouldn't do this unless we have a demonstrated need. Though, having said that, this change doesn't hurt anything.

If it's not needed then just lets not spend time on it.