Page MenuHomePhabricator

[IPSCCP] Move the IPSCCP run function under the IPO directory.
Needs ReviewPublic

Authored by labrinea on Thu, Nov 24, 4:15 AM.

Details

Summary

The LLVMipo library no longer depends on the Scalar component. The shared functions between IPSCCP and SCCP have been moved under Utils, in the SCCPSolver.

This is preliminary work for D126455, in order to break cyclic dependecy of LLVM libraries.

Diff Detail

Unit TestsFailed

TimeTest
60,040 msx64 debian > libFuzzer.libFuzzer::minimize_crash.test
Script: -- : 'RUN: at line 1'; /var/lib/buildkite-agent/builds/llvm-project/build/./bin/clang --driver-mode=g++ -O2 -gline-tables-only -fsanitize=address,fuzzer -I/var/lib/buildkite-agent/builds/llvm-project/compiler-rt/lib/fuzzer -m64 /var/lib/buildkite-agent/builds/llvm-project/compiler-rt/test/fuzzer/NullDerefTest.cpp -o /var/lib/buildkite-agent/builds/llvm-project/build/projects/compiler-rt/test/fuzzer/X86_64DefaultLinuxConfig/Output/minimize_crash.test.tmp-NullDerefTest

Event Timeline

labrinea created this revision.Thu, Nov 24, 4:15 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Nov 24, 4:15 AM
labrinea requested review of this revision.Thu, Nov 24, 4:15 AM
Herald added a project: Restricted Project. · View Herald TranscriptThu, Nov 24, 4:15 AM
labrinea added inline comments.Thu, Nov 24, 4:18 AM
llvm/lib/Transforms/Scalar/SCCP.cpp
61–64

Some of these were used in IPSCCP, but it seems wrong to me. Now that I've separated SCCP from IPSCCP, I've duplicated any shared statistics.