Page MenuHomeFreeBSD

powerpc: Enable LLDB on all powerpc architectures
ClosedPublic

Authored by pkubaj on Jan 25 2022, 10:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 14, 7:35 AM
Unknown Object (File)
Thu, May 9, 3:44 PM
Unknown Object (File)
Apr 27 2024, 2:39 PM
Unknown Object (File)
Apr 27 2024, 2:33 PM
Unknown Object (File)
Apr 27 2024, 1:38 PM
Unknown Object (File)
Apr 27 2024, 1:06 PM
Unknown Object (File)
Apr 24 2024, 8:06 AM
Unknown Object (File)
Feb 21 2024, 4:30 AM

Details

Summary

While LLDB on powerpc and powerpcspe builds as-is, on powerpc64 and
powerpc64le it requires adding a couple of additional source files
to build.

Test Plan

make universe
try running lldb on target systems

Diff Detail

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

Event Timeline

One nit that's more an out loud question than a required change.

share/mk/src.opts.mk
285

so only armv and riscv are broken? Maybe it's time to reverse the sense of this conditional?

This revision is now accepted and ready to land.Jan 25 2022, 11:16 PM
emaste added inline comments.
share/mk/src.opts.mk
285

I suspect that 32-bit arm is fine, so it could probably be just != riscv. In any case though I think we can do it as a separate change.