powerpc64*: workaround for broken binaries generated with llvm12
AbandonedPublic
Actions

Authored by alfredo on Aug 23 2021, 2:05 PM.

Details

Reviewers

jhibbits
bdragon
dim
luporl

Summary

powerpc64 and powerpc64 binaries are crashing with segmentation fault when -fstack-protector-strong is used with any optimization level. The binaries seem correct if one of -fstack-protector or -O[0|1|2|3] is ommited.

This patch add a temporary workaround while root cause is being investigated.

MFC after: 1 day

Test Plan

buildworld + chroot
run binaries like clang, ld.lld, llvm-objdump, ls, cp, echo

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 41168
Build 38057: arc lint + arc unit

Event Timeline

alfredo created this revision.Aug 23 2021, 2:05 PM

Herald added subscribers: luporl, emaste, imp, bdrewery. · View Herald TranscriptAug 23 2021, 2:05 PM

alfredo requested review of this revision.Aug 23 2021, 2:05 PM

alfredo edited reviewers, added: luporl; removed: leandro.lupori_gmail.com.

Harbormaster completed remote builds in B41167: Diff 94054.Aug 23 2021, 2:06 PM

is there a PR to reference here?

Move logic to BROKEN_OPTIONS as suggested by @jhibbits

Harbormaster completed remote builds in B41168: Diff 94055.Aug 23 2021, 2:45 PM

In D31646#713717, @emaste wrote:

is there a PR to reference here?

A PR against LLVM? Not yet. I reduced it to a test program available at https://people.freebsd.org/~alfredo/tmp/llvm12_crash/. I should file a PR later today.
On FreeBSD side there a report from a user on freebsd-ppc mailing list: https://lists.freebsd.org/archives/freebsd-ppc/2021-August/000035.html

Move logic to BROKEN_OPTIONS as suggested by @jhibbits

This will work for src itself, but I think it won't work with ports that use /usr/share/mk?

In D31646#713731, @emaste wrote:

Move logic to BROKEN_OPTIONS as suggested by @jhibbits

This will work for src itself, but I think it won't work with ports that use /usr/share/mk?

I talked to @pkubaj on IRC, he should take care of ports side after this patch is commited

Abandoned in favor of D31698

Revision Contents
Changeset List

Path

Size

share/

mk/

src.opts.mk

5 lines

Diff 94055

View Options

powerpc64*: workaround for broken binaries generated with llvm12AbandonedPublicActions