Page MenuHomeFreeBSD

[PowerPC64] LLVM as default toolchain for PowerPC and PowerPC64
Changes PlannedPublic

Authored by alfredo.junior_eldorado.org.br on May 23 2019, 2:09 PM.

Details

Reviewers
jhibbits
luporl
Summary

Warning: to be commited on "flag day", so marking as "Plan Changes"

This enables LLVM as default toolchain/compiler/linker for PowerPC and PowerPC64 platforms.

Diff Detail

Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 25243
Build 23912: arc lint + arc unit

Event Timeline

alfredo.junior_eldorado.org.br planned changes to this revision.

All known outstanding issues are recorded as parents of this review?

share/mk/src.opts.mk
272

Should this part be committed independently now? (I.e., this should be applied regardless of what the default is?)

326

I suppose we cannot turn LLVM_LIBUNWIND on in advance (with gcc 4.2.1)

share/mk/src.opts.mk
272

Actually this is just a preparation for the flag day, marking this as "Plan Changes" to avoid accidental commit.
This shouldn't be commited now, but reviews are welcome.

326

True. This need to wait for the flag day.

emaste added inline comments.May 23 2019, 6:31 PM
share/mk/src.opts.mk
272

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.

alfredo.junior_eldorado.org.br marked 3 inline comments as done.May 23 2019, 7:19 PM
alfredo.junior_eldorado.org.br added inline comments.
share/mk/src.opts.mk
272

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.
If you prefer to split this part, just let me know.

alfredo.junior_eldorado.org.br marked an inline comment as done.Tue, Jun 25, 2:07 PM

Moved _LLVM_TARGET_FILT change to D20756

alfredo.junior_eldorado.org.br planned changes to this revision.Tue, Jun 25, 3:23 PM

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.

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.

Sure, this would require changes in D20261 as well since base LLD is not ready

alfredo.junior_eldorado.org.br retitled this revision from [PowerPC64] LLVM is now the default toolchain for PowerPC64 target binutils BFD linker is kept for compiling 32 bit binaries such as LIB32 and stand/slof to [PowerPC64] LLVM as default toolchain for PowerPC and PowerPC64.Mon, Jul 8, 12:37 PM
alfredo.junior_eldorado.org.br edited the summary of this revision. (Show Details)
alfredo.junior_eldorado.org.br updated this revision to Diff 59532.
  • Includes PowerPC 32 bit in the list.
  • removes LLDB for PowerPC64, since it expects "powerpc64le" and we are in BE.
alfredo.junior_eldorado.org.br planned changes to this revision.Mon, Jul 8, 12:38 PM