This is an archive of the discontinued LLVM Phabricator instance.

add testcases for PR30188
Needs ReviewPublic

Authored by sebpop on Sep 1 2016, 12:06 PM.

Details

Summary

Add testcases to llvm test/ to make sure that the combined middle-end
optimizations do not regress on optimizing the number of loads in the loops.

Diff Detail

Event Timeline

sebpop updated this revision to Diff 70045.Sep 1 2016, 12:06 PM
sebpop retitled this revision from to add testcases for PR30188.
sebpop updated this object.
sebpop added reviewers: jmolloy, danielcdh, davidxl.
sebpop added a subscriber: llvm-commits.
spatel added a subscriber: spatel.Sep 2 2016, 2:11 PM

I thought we frown on multi-pass regression tests, let alone -O2.

Based on the comments in PR30188 ( https://llvm.org/bugs/show_bug.cgi?id=30188 ) - if the goal is to make sure that BinarySearch perf does not regress, is a better solution to add that C source app to test-suite's Benchmarks dir, so we can make sure that the end-to-end perf is what we want?