Skip to content

sz: Change optimization flags for Fujitsu compiler#14007

Merged
adamjstewart merged 1 commit intospack:developfrom
t-karatsu:bugfix/sz
Dec 6, 2019
Merged

sz: Change optimization flags for Fujitsu compiler#14007
adamjstewart merged 1 commit intospack:developfrom
t-karatsu:bugfix/sz

Conversation

@t-karatsu
Copy link
Copy Markdown
Contributor

@t-karatsu t-karatsu commented Dec 6, 2019

Part of latest sources couldn't execute to optimize -O3 with Fujitsu compiler.
I'll lower optimize level to -O2 only for Fujitsu compiler.

References for similar bugs:
HexHive/CFIXX#6

Real error log:

libtool: compile:  /~/spack/lib/spack/env/fj/fcc -DHAVE_CONFIG_H -I. -I.. -I./ -I./compress -I./common -I./deprecated -I./dictBuilder -I./legacy -O3 -std=c99 -Wall -MT ./legacy/libzstd_la-zstd_v02.lo -MD -MP -MF ./legacy/.deps/libzstd_la-zstd_v02.Tpo -c ./legacy/zstd_v02.c  -fPIC -DPIC -o ./legacy/.libs/libzstd_la-zstd_v02.o

/~/lib64/libLLVMSupport.so.6(_ZN4ll
vm3sys15PrintStackTraceERNS_11raw_ostreamE+0x28)[0xffff9fd20888]
Stack dump:
0.      Program arguments: ~
1.      <eof> parser at end of file
2.      Per-module optimization passes
3.      Running pass 'CallGraph Pass Manager' on module './compress/zstd_opt.c'.
4.      Running pass 'Loop Pass Manager' on function '@ZSTD_compressBlock_opt_generic'
5.      Running pass 'Tree Height Reduction' on basic block '%7049'
libtool: compile:  /~/spack/lib/spack/env/fj/fcc -DHAVE_CONFIG_H -I. -I.. -I./ -I./compress -I./common -I./deprecated -I./dictBuilder -I./legacy -O3 -std=c99 -Wall -MT ./compress/libzstd_la-zstd_compress.lo -MD -MP -MF ./compress/.deps/libzstd_la-zstd_compress.Tpo -c ./compress/
zstd_compress.c -o ./compress/libzstd_la-zstd_compress.o >/dev/null 2>&1
...

@adamjstewart adamjstewart merged commit 246799d into spack:develop Dec 6, 2019
@t-karatsu t-karatsu deleted the bugfix/sz branch December 12, 2019 04:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants