**Warning: to be commited on "flag day"
This enables LLVM as default toolchain/compiler/linker for PowerPC, PowerPCSPE and PowerPC64 platforms.
Diff Detail
Diff Detail
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 27439 Build 25677: arc lint + arc unit
Event Timeline
| share/mk/src.opts.mk | ||
|---|---|---|
| 281 | I mean, it seems this specific __LLVM_TARGET_FILT change is a bug fix that should go in before the flag day? Any use of Clang on powerpc64 is inherently not the default but needs this change? It doesn't really matter if this doesn't go in until later, just something I wondered about. | |
| share/mk/src.opts.mk | ||
|---|---|---|
| 281 | oh yeah, this is bug fix! I guess it could go in before, but I personally didn't verified for undesired side effects with current default ABI. | |
Comment Actions
Can't this be done for powerpc*? powerpcspe is probably the only thing that doesn't fully work upstream yet, but I have patches in-flight that might land in LLVM9 before too long.
Comment Actions
- Includes PowerPC 32 bit in the list.
- removes LLDB for PowerPC64, since it expects "powerpc64le" and we are in BE.
Comment Actions
Patch updated to resolve conflict with current HEAD, and also makes LLD as LD in powerpc (thanks to @bdragon)
Comment Actions
ObsoleteFiles.inc will also need updates since we switch from ld.bfd for 32-bits after all.
Comment Actions
You will also need updates to share/man/man7/arch.7 to account for default compiler and linker changes.
Comment Actions
AFAIK the plan is to switch all three powerpc* at once and there are currently problems on powerpc64 with linuxkpi (https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=241728) and on powerpc with PF kernel module.
| share/man/man7/arch.7 | ||
|---|---|---|
| 347 | minor nit: "mips or sparc64" is a bit tidier than "mips, nor sparc64", English wise. | |