Issue 350 in webp: [cmake] use of -mno-sse2 with clang-3.9.1 causes an ICE

[jz… via monorail]

Status: Accepted
Owner: jz...@google.com
CC: vincent....@gmail.com
Labels: Type-Defect Priority-Medium

New issue 350 by jz...@google.com: [cmake] use of -mno-sse2 with clang-3.9.1 causes an ICE
https://bugs.chromium.org/p/webp/issues/detail?id=350

v0.6.0-130-g28914528

$ cmake .. -DCMAKE_C_COMPILER=clang -DWEBP_ENABLE_SIMD=OFF

variously, depending on the module:

fatal error: error in backend: SSE2 register return with SSE2 disabled

or

1. <eof> parser at end of file
2. Code generation
3. Running pass 'Function Pass Manager' on module '../src/enc/picture_csp_enc.c'.
4. Running pass 'X86 FP Stackifier' on function '@ImportYUVAFromRGBA'
clang-3.9: error: unable to execute command: Segmentation fault
clang-3.9: error: clang frontend command failed due to signal (use -v to see invocation)
clang version 3.9.1 (tags/RELEASE_391/final 293390)


--
You received this message because:
1. The project was configured to send all issue notifications to this address

You may adjust your notification preferences at:
https://bugs.chromium.org/hosting/settings

[jz… via monorail]

Comment #1 on issue 350 by jz...@google.com: [cmake] use of -mno-sse2 with clang-3.9.1 causes an ICE
https://bugs.chromium.org/p/webp/issues/detail?id=350#c1

Options include:

- change the code to respect the presence of HAVE_CONFIG_H and !defined(WEBP_HAVE_SSE2) over the presence of __SSE2__ when defining WEBP_USE_SSE2. --disable-sse2 to configure is non-functional with toolchains defaulting to sse2+ for this reason.

- check the clang version and drop the flag, that would be the same as configure currently

- ignore the ICE if clang-4.0 etc fixed the issue

[bugdro… via monorail]

Updates:
Labels: merge-merged-portable-intrinsics

Comment #2 on issue 350 by bugd...@chromium.org: [cmake] use of -mno-sse2 with clang-3.9.1 causes an ICE
https://bugs.chromium.org/p/webp/issues/detail?id=350#c2

The following revision refers to this bug:
https://chromium.googlesource.com/webm/libwebp/+/a604ab5600dd185ba783d7ca7fef4a8d91eb1a33

commit a604ab5600dd185ba783d7ca7fef4a8d91eb1a33
Author: James Zern <jz...@google.com>
Date: Thu Jun 29 02:34:24 2017

cpu.cmake: skip simd disable flag check w/wasm

this workarounds an ICE with clang. WASM overrides the native simd so
apart from binary size building it isn't an issue.

BUG=webp:350

Change-Id: Ib0195049249e6cb86d3225ce5db7247ac22cdbd6

[modify] https://crrev.com/a604ab5600dd185ba783d7ca7fef4a8d91eb1a33/cmake/cpu.cmake

[vrab… via monorail]

Comment #3 on issue 350 by vra...@google.com: [cmake] use of -mno-sse2 with clang-3.9.1 causes an ICE
https://bugs.chromium.org/p/webp/issues/detail?id=350#c3

jzern@, isn't that fixed by your commit? I cannot replicate locally.

[jz… via monorail]

Comment #4 on issue 350 by jz...@google.com: [cmake] use of -mno-sse2 with clang-3.9.1 causes an ICE
https://bugs.chromium.org/p/webp/issues/detail?id=350#c4

It's fixed in that a workaround is in place, but I think absent that you could still get an ICE on master.

[jz… via monorail]

Updates:
Status: WontFix

Comment #5 on issue 350 by jz...@google.com: [cmake] use of -mno-sse2 with clang-3.9.1 causes an ICE
https://bugs.chromium.org/p/webp/issues/detail?id=350#c5

Going to mark this wontfix. It's not a common option and can be worked around.