Page MenuHomeFreeBSD

clang: enable the LoongArch target in in-tree llvm builds
Needs ReviewPublic

Authored by zhaoxiaoqiang007_gmail.com on Tue, Sep 22, 2:25 PM.
Tags
None
Referenced Files
F173244574: D59899.id.diff
Thu, Sep 24, 3:51 PM
F173208977: D59899.diff
Thu, Sep 24, 7:46 AM
Unknown Object (File)
Wed, Sep 23, 8:41 AM
Unknown Object (File)
Wed, Sep 23, 12:17 AM
Unknown Object (File)
Tue, Sep 22, 11:44 PM
Unknown Object (File)
Tue, Sep 22, 10:16 PM
Unknown Object (File)
Tue, Sep 22, 6:44 PM
Unknown Object (File)
Tue, Sep 22, 3:03 PM

Details

Reviewers
dim
emaste
jkoshy
Summary

Enable the LoongArch target, assembler parser and disassembler.

Signed-off-by: Xiaoqiang Zhao <zhaoxiaoqiang007@gmail.com>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 77179
Build 74062: arc lint + arc unit

Event Timeline

Do we need a change in share/mk/src.opts.mk? Adding LLVM_TARGET_LOONGARCH to __DEFAULT_NO_OPTIONS for now?

src.conf.5 will need to be regenerated as well

I'm generally fine with this, but it is there a concrete plan to add LoongArch support for the base system? Otherwise I would advise to use one of the llvm ports for bringup first.

Btw, this would also need adjustments in share/mk/src.opts.mk to add the MK_LLVM_TARGET_LOONGARCH make macro.

Do we need a change in share/mk/src.opts.mk? Adding LLVM_TARGET_LOONGARCH to __DEFAULT_NO_OPTIONS for now?

src.conf.5 will need to be regenerated as well

Disable it for now is make sense

is there a concrete plan to add LoongArch support for the base system?

There's a lot of work (in review) to add support. I suspect we'll be iterating on it for some time, but IMO it's reasonable to have the toolchain infrastructure in the base system now.

is there a concrete plan to add LoongArch support for the base system?

There's a lot of work (in review) to add support. I suspect we'll be iterating on it for some time, but IMO it's reasonable to have the toolchain infrastructure in the base system now.

agree, as it already works well

I just removed the remnants of the Mips target in rGb5d1e0c5a4929e2be3bc58aad8e5b707860ec0fd, please rebase.

In D59899#1376284, @dim wrote:

I just removed the remnants of the Mips target in rGb5d1e0c5a4929e2be3bc58aad8e5b707860ec0fd, please rebase.

got it