Opened 3 years ago

Closed 3 years ago

#62876 closed defect (fixed)

simde: port fails to build for Intel with default variant +tests enabled

Reported by: mascguy (Christopher Nielsen) Owned by: jasonliu-- (Jason Liu)
Priority: Normal Milestone:
Component: ports Version:
Keywords: Cc:
Port: simde

Description

I'd love to try utilizing simde for ports like gerbil, which directly depend on Intel intrinsics.

The problem is, it fails to build on Intel. Thus far, I've only tried building on MacOS 10.13, but this is what I'm seeing:

:info:build [631/1748] ccache /usr/bin/clang -Itest/x86/sse-native-c.p -Itest/x86 -I../simd-everywhere-simde-12069d7/test/x86 -I. 
-I../simd-everywhere-simde-12069d7 -I/opt/local/include -Xclang -fcolor-diagnostics -pipe -Wall -Winvalid-pch -std=c99 -g -Os -arc
h x86_64 -fopenmp-simd -DSIMDE_ENABLE_OPENMP -DSIMDE_TEST_BARE -MD -MQ test/x86/sse-native-c.p/sse.c.o -MF test/x86/sse-native-c.p
/sse.c.o.d -o test/x86/sse-native-c.p/sse.c.o -c ../simd-everywhere-simde-12069d7/test/x86/sse.c
:info:build FAILED: test/x86/sse-native-c.p/sse.c.o 
:info:build ccache /usr/bin/clang -Itest/x86/sse-native-c.p -Itest/x86 -I../simd-everywhere-simde-12069d7/test/x86 -I. -I../simd-e
verywhere-simde-12069d7 -I/opt/local/include -Xclang -fcolor-diagnostics -pipe -Wall -Winvalid-pch -std=c99 -g -Os -arch x86_64 -f
openmp-simd -DSIMDE_ENABLE_OPENMP -DSIMDE_TEST_BARE -MD -MQ test/x86/sse-native-c.p/sse.c.o -MF test/x86/sse-native-c.p/sse.c.o.d 
-o test/x86/sse-native-c.p/sse.c.o -c ../simd-everywhere-simde-12069d7/test/x86/sse.c
:info:build clang: error: unable to execute command: Segmentation fault: 11
:info:build clang: error: clang frontend command failed due to signal (use -v to see invocation)

Of note, the port installs just fine without variant +tests enabled, as no compilation is necessary.

So I'm wondering if that variant shouldn't be enabled by default...?

Attachments (1)

simde-build-failure-10.13.txt.gz (17.3 KB) - added by mascguy (Christopher Nielsen) 3 years ago.

Download all attachments as: .zip

Change History (10)

Changed 3 years ago by mascguy (Christopher Nielsen)

comment:1 Changed 3 years ago by mascguy (Christopher Nielsen)

Builds are failing on MacOS 10.12, with the same error.

It looks like the CI builds succeeded for this new port, via the PR. So perhaps builds succeed for 10.14+. But if we're going to keep +tests enabled by default, the build needs to be fixed for older releases.

Or the simpler approach is to not enable that variant by default. Up to you.

comment:2 Changed 3 years ago by jasonliu-- (Jason Liu)

Cc: jasonliu-- added

comment:3 Changed 3 years ago by jasonliu-- (Jason Liu)

Cc: jasonliu-- removed

comment:4 Changed 3 years ago by jasonliu-- (Jason Liu)

Very interesting... I saw a similar problem on the sse2neon port. Did you build the port locally by downloading the portfile? If so, could you try changing the the blacklisted clang version from 802 to 1001 and see whether it builds on 10.13 / 10.12?

Last edited 3 years ago by jasonliu-- (Jason Liu) (previous) (diff)

comment:5 in reply to:  4 Changed 3 years ago by mascguy (Christopher Nielsen)

Replying to jasonliu--:

Very interesting... I saw a similar problem on the sse2neon port. Did you build the port locally by downloading the portfile? If so, could you try changing the the blacklisted clang version from 802 to 1001 and see whether it builds on 10.13 / 10.12?

That fixed the issue for 10.12, but still need to test on 10.13.

comment:6 Changed 3 years ago by jasonliu-- (Jason Liu)

That's good news. Let me know if it also fixes the issue on 10.13, and I can submit a PR for the fix.

comment:7 Changed 3 years ago by mascguy (Christopher Nielsen)

The fix works for 10.13 as well.

Let me know when you've submitted a PR, and I'll merge.

comment:8 Changed 3 years ago by jasonliu-- (Jason Liu)

comment:9 Changed 3 years ago by mascguy (Christopher Nielsen)

Resolution: fixed
Status: assignedclosed

Changes merged; closing as fixed.

Note: See TracTickets for help on using tickets.