Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Polly] -polly-parallel miscompiles two LNT benchmarks #26808

Closed
tobiasgrosser opened this issue Feb 2, 2016 · 3 comments
Closed

[Polly] -polly-parallel miscompiles two LNT benchmarks #26808

tobiasgrosser opened this issue Feb 2, 2016 · 3 comments
Labels
bugzilla Issues migrated from bugzilla polly

Comments

@tobiasgrosser
Copy link
Contributor

Bugzilla Link 26434
Resolution FIXED
Resolved on Feb 07, 2016 06:50
Version unspecified
OS Linux

Extended Description

We see currently two miscompiles in LNT with -polly-parallel:

http://lab.llvm.org:8011/builders/perf-x86_64-penryn-O3-polly-parallel-fast/builds/15119

@llvmbot
Copy link
Collaborator

llvmbot commented Feb 4, 2016

I’m not able to reproduce them on my target machine. I’ve run the LNT with the following options:

~/mysandbox/bin/lnt runtest nt --sandbox SANDBOX --cc "/home/roman/Documents/polly/build/llvm/bin/clang" --cxx "/home/roman/Documents/polly/build/llvm/bin/clang++"  --cflags=" -lgomp -Xclang -load -Xclang /tmp_home/compiled/llvm/lib/LLVMPolly.so -O3 -mllvm -polly -mllvm -polly-parallel" --test-suite ~/llvm-test-suite

Have I missed something?

@tobiasgrosser
Copy link
Contributor Author

I had issues reproducing them as well. So no, this does not seem to be an easy/simple to debug issue.

@tobiasgrosser
Copy link
Contributor Author

This was likely not a miscompile, but rather too many threads running in parallel as we run LNT with -j16 and then allowed each program to create itself threads without limitation (which resulted in one thread per core and process). This issue was resolved by hardcoding the number of parallel threads per process with OMP_NUM_THREADS=2.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla polly
Projects
None yet
Development

No branches or pull requests

2 participants