Merge PR #188: Fix dynamic metrics tool on core-lib/Benchmarks/All.ns

smarr · Oct 6, 2017 · c3e8381 · c3e8381
2 parents 4cb272e + 992ffbd
commit c3e8381
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/core-lib/Benchmarks/LanguageFeatures.ns b/core-lib/Benchmarks/LanguageFeatures.ns
@@ -386,7 +386,7 @@ class LanguageFeaturesSuite usingPlatform: platform andHarness: harness = (
     singleRun = (
         | sum v |
         v:: Vector new.
-        v at: 1 put: 1.
+        v append: 1.
         sum:: 0.
         [sum < 1000]
             whileTrue:

diff --git a/src/tools/dym/MetricsCsvWriter.java b/src/tools/dym/MetricsCsvWriter.java
@@ -217,7 +217,8 @@ private String operationType(final OperationProfile p, final Arguments a) {
           p.getOperation().equals("sqrt") ||
           p.getOperation().equals("sin") ||
           p.getOperation().equals("cos") ||
-          p.getOperation().equals("asInteger")) {
+          p.getOperation().equals("asInteger") ||
+          p.getOperation().equals("round")) {
         return typeCategory(a.getArgType(1));
       } else if (p.getOperation().equals("as32BitUnsignedValue") ||
           p.getOperation().equals("as32BitSignedValue")) {

diff --git a/tests/dym/expected-results.tar.bz2 b/tests/dym/expected-results.tar.bz2