-
Notifications
You must be signed in to change notification settings - Fork 16
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
NodePushOut causes 10 failed tests in llvm-test-suite #641
Comments
Enabling fewer optimizations, leads to more failed tests: set(OPTFLAGS "${OPTFLAGS} -JInvariantValueRedirection -JNodeReduction -JDeadNodeElimination -JThetaGammaInversion -JInvariantValueRedirection -JDeadNodeElimination -JNodePushOut -JInvariantValueRedirection -JDeadNodeElimination -JNodeReduction -JCommonNodeElimination ") Failed Tests (14): |
set(OPTFLAGS "${OPTFLAGS} -JNodePushOut -JInvariantValueRedirection -JDeadNodeElimination -JNodeReduction -JCommonNodeElimination") Failed Tests (10): |
-JNodePushOut -JCommonNodeElimination cause extreme compilation time. time
|
The NodePushOut optimization was improved in association with addressing #632 and the optimization in isolation now passes the llvm-test-suite.
However, when including it in the standard 'opt.cmake' that is used by the CI then 10 tests fails:
set(OPTFLAGS "${OPTFLAGS} -JInvariantValueRedirection -JNodeReduction -JDeadNodeElimination -JThetaGammaInversion -JInvariantValueRedirection -JDeadNodeElimination -JNodePushOut -JInvariantValueRedirection -JDeadNodeElimination -JNodeReduction -JCommonNodeElimination -JDeadNodeElimination -JNodePullIn -JInvariantValueRedirection -JDeadNodeElimination -JLoopUnrolling -JInvariantValueRedirection")
Failed Tests (10):
test-suite :: MultiSource/Applications/JM/ldecod/ldecod.test
test-suite :: MultiSource/Applications/lua/lua.test
test-suite :: MultiSource/Applications/sqlite3/sqlite3.test
test-suite :: MultiSource/Benchmarks/FreeBench/analyzer/analyzer.test
test-suite :: MultiSource/Benchmarks/MallocBench/espresso/espresso.test
test-suite :: SingleSource/Regression/C/gcc-c-torture/execute/GCC-C-execute-20020619-1.test
test-suite :: SingleSource/Regression/C/gcc-c-torture/execute/GCC-C-execute-pr37882.test
test-suite :: SingleSource/Regression/C/gcc-c-torture/execute/GCC-C-execute-pr40747.test
test-suite :: SingleSource/Regression/C/gcc-c-torture/execute/GCC-C-execute-pr57568.test
test-suite :: SingleSource/Regression/C/gcc-c-torture/execute/GCC-C-execute-pr84339.test
The text was updated successfully, but these errors were encountered: