Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

Commit

Permalink
Update SpanBench test
Browse files Browse the repository at this point in the history
 - Re-enable the tests that were disabled in e859c30.
 - Re-work how tests are invoked from the command line to require less
   boilerplate
     - Now each test has a single [Benchmark] entrypoint
     - That entrypoint invokes its test's single inner-loop by wrapping it
       in a lambda that it passes to a new helper Invoke method (shared
       across all tests) which handles the xunit vs. command-line
       differences..
     - Main finds the entrypoints by using reflection to search for the
       [Benchmark] attributes, so the explicit list of stringified test
       names is no longer needed and the command line will run the same
       set of tests that xunit-perf does.
 - The new SpanAPI tests now get invoked when this tests is run from the
   command line as well.
 - Add [NoInlining] to the SpanAPI tests' kernels.
 - Add some heap writes and conditional writes to prevent deadcode
   optimization from eliminating tests' kernels (assuming we don't do
   interprocedural constant propagation or deadcode, or store sinking and
   final value calculation, or PRE...).
 - Split the Index SpanAPI tests into one version that uses a
   loop-invariant index and thus should get hoisted out of the loop, and
   another version that uses variant indices.
  • Loading branch information
JosephTremoulet committed Mar 18, 2017
1 parent c1f3865 commit 65226a2
Show file tree
Hide file tree
Showing 2 changed files with 719 additions and 519 deletions.
1 change: 1 addition & 0 deletions tests/src/Common/test_dependencies/project.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
"dependencies": {
"Microsoft.NETCore.Platforms": "2.0.0-beta-25118-01",
"Microsoft.Private.CoreFx.NETCoreApp": "4.4.0-beta-25118-01",
"System.Memory": "4.4.0-beta-25118-01",
"System.Runtime.CompilerServices.Unsafe": "4.4.0-beta-25118-01"
},
"frameworks": {
Expand Down
Loading

0 comments on commit 65226a2

Please sign in to comment.