From 03412b9f841f140b9b7410a1890298e3ac2835db Mon Sep 17 00:00:00 2001
From: Joshua Bakita <jbakita@cs.unc.edu>
Date: Tue, 20 Oct 2020 13:35:52 -0400
Subject: TACLe and DIS benchmark fixes

- Use consistent approach to prevent computations from being
  optimized out in the DIS benchmarks
- Don't print results in the DIS benchmarks
- Fix a memory corruption in TACLe's "epic" benchmark reflect1()
- Fix return codes for all benchmarks
- Rename run_baseline.sh to run_bench.sh in preperation for this
  being the main benchmarking script
---
 dis/Field/field.c               | 14 +++++++-------
 dis/Matrix/ver2/matrix.c        |  6 ++++--
 dis/Neighborhood/neighborhood.c | 33 ++++++++++++++++++++-------------
 dis/Pointer/pointer.c           |  9 +++++----
 dis/Transitive/transitive.c     | 10 +++++-----
 dis/Update/update.c             |  4 ++--
 6 files changed, 43 insertions(+), 33 deletions(-)

(limited to 'dis')

diff --git a/dis/Field/field.c b/dis/Field/field.c
index 900d47b..d028ee7 100644
--- a/dis/Field/field.c
+++ b/dis/Field/field.c
@@ -137,17 +137,17 @@ int main(int argc, char **argv) {
   }
   endTime = time(NULL);
 
-  volatile int sumAll = 0;
+  volatile int _stop_optimizer = 0;
   for (l = 0; l < n; l++) {
     unsigned int ll;
-    // fprintf(stdout, "%d subfields for token %d \n", token[l].subfields, l);
-    sumAll += token[l].subfields + l;
+    //fprintf(stdout, "%d subfields for token %d \n", token[l].subfields, l);
+    _stop_optimizer += token[l].subfields + l;
     for (ll = 0; ll < token[l].subfields; ll++) {
-      sumAll += ll + token[l].stat[ll].count + token[l].stat[ll].min +
+      _stop_optimizer += ll + token[l].stat[ll].count + token[l].stat[ll].min +
                 token[l].stat[ll].sum;
-      /*fprintf(stdout, "subfields %d:\tcount = %d\tmin= %x\tsum= %x\n",
-          ll, token[l].stat[ll].count,
-          token[l].stat[ll].min, token[l].stat[ll].sum);*/
+      //fprintf(stdout, "subfields %d:\tcount = %d\tmin= %x\tsum= %x\n",
+      //    ll, token[l].stat[ll].count,
+      //    token[l].stat[ll].min, token[l].stat[ll].sum);*/
     }
   }
   fprintf(stderr, "time for field stressmark = %f seconds.\n",
diff --git a/dis/Matrix/ver2/matrix.c b/dis/Matrix/ver2/matrix.c
index 5162579..9d9ac7e 100755
--- a/dis/Matrix/ver2/matrix.c
+++ b/dis/Matrix/ver2/matrix.c
@@ -551,8 +551,10 @@ int main(int argc, char **argv) {
     sum += sum + *(vectorX + k);
   }
 
-  fprintf(stdout, "sum = %f, actualError = %e, actualIteration = %d\n", sum,
-          actualError, actualIteration);
+  //fprintf(stdout, "sum = %f, actualError = %e, actualIteration = %d\n", sum,
+  //        actualError, actualIteration);
+  volatile double _stop_optimizer = sum + actualError + actualIteration;
+
   fprintf(stderr, "time for matrix stressmark = %f seconds.\n",
           difftime(endTime, beginTime));
 
diff --git a/dis/Neighborhood/neighborhood.c b/dis/Neighborhood/neighborhood.c
index 2d401d3..ea1c658 100644
--- a/dis/Neighborhood/neighborhood.c
+++ b/dis/Neighborhood/neighborhood.c
@@ -74,23 +74,30 @@ int main(int argc, char **argv) {
   endTime = time(NULL);
   WRITE_TO_FILE
 
-  //  printf(" end time is %d\n", endTime);
+  volatile double _stop_optimizer = values.distShort.deg0.entropy
+          + values.distShort.deg0.energy + values.distShort.deg45.entropy
+          + values.distShort.deg45.energy + values.distShort.deg90.entropy
+          + values.distShort.deg90.energy + values.distShort.deg135.entropy
+          + values.distShort.deg135.energy + values.distLong.deg0.entropy
+          + values.distLong.deg0.energy + values.distShort.deg45.entropy
+          + values.distLong.deg45.energy + values.distLong.deg90.entropy
+          + values.distLong.deg90.energy + values.distLong.deg135.entropy
+          + values.distLong.deg135.energy;
+  //fprintf(stdout, "%9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e",
+  //        values.distShort.deg0.entropy, values.distShort.deg0.energy,
+  //        values.distShort.deg45.entropy, values.distShort.deg45.energy,
+  //        values.distShort.deg90.entropy, values.distShort.deg90.energy,
+  //        values.distShort.deg135.entropy, values.distShort.deg135.energy);
 
-  fprintf(stdout, "%9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e",
-          values.distShort.deg0.entropy, values.distShort.deg0.energy,
-          values.distShort.deg45.entropy, values.distShort.deg45.energy,
-          values.distShort.deg90.entropy, values.distShort.deg90.energy,
-          values.distShort.deg135.entropy, values.distShort.deg135.energy);
-
-  fprintf(stdout, "%9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e",
-          values.distLong.deg0.entropy, values.distLong.deg0.energy,
-          values.distShort.deg45.entropy, values.distLong.deg45.energy,
-          values.distLong.deg90.entropy, values.distLong.deg90.energy,
-          values.distLong.deg135.entropy, values.distLong.deg135.energy);
+  //fprintf(stdout, "%9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e %9.4e",
+  //        values.distLong.deg0.entropy, values.distLong.deg0.energy,
+  //        values.distShort.deg45.entropy, values.distLong.deg45.energy,
+  //        values.distLong.deg90.entropy, values.distLong.deg90.energy,
+  //        values.distLong.deg135.entropy, values.distLong.deg135.energy);
 
   fprintf(stderr, "time for neghborhood stressmark = %f seconds.\n",
           difftime(endTime, beginTime));
 
   free((Pixel *)image);
-  return (SUCCESS);
+  return (0);
 }
diff --git a/dis/Pointer/pointer.c b/dis/Pointer/pointer.c
index d97f276..0325c64 100644
--- a/dis/Pointer/pointer.c
+++ b/dis/Pointer/pointer.c
@@ -146,10 +146,11 @@ int main(int argc, char **argv) {
 
   endTime = time(NULL);
 
-  volatile int _stop_optimizer = thread[l].hops + l;
-  /*for (l=0; l<n; l++){
-   fprintf(stdout, "%lu hops on thread %d\n", thread[l].hops, l);
-  }*/
+  volatile int _stop_optimizer = 0;
+  for (l=0; l<n; l++){
+      _stop_optimizer += thread[l].hops + l;
+  //  fprintf(stdout, "%lu hops on thread %d\n", thread[l].hops, l);
+  }
 
   fprintf(stderr, "time for pointer stressmark = %f seconds.\n",
           difftime(endTime, startTime));
diff --git a/dis/Transitive/transitive.c b/dis/Transitive/transitive.c
index 9337aad..0679f7e 100644
--- a/dis/Transitive/transitive.c
+++ b/dis/Transitive/transitive.c
@@ -47,7 +47,6 @@ int main(int argc, char **argv) {
 
   time_t startTime, stopTime;
   unsigned int sum;
-  volatile int realRes = 0;
 
   assert(fscanf(stdin, "%d %d %d", &n, &m, &seed) == 3);
 
@@ -100,14 +99,15 @@ int main(int argc, char **argv) {
 
   stopTime = time(NULL);
 
+  volatile int _stop_optimizer = 0;
   for (j = 0; j < n; j++) {
     sum = 0;
     for (i = 0; i < n; i++) {
       if (*(din + j * n + i) != NO_PATH)
         sum += *(din + j * n + i);
     }
-    realRes += sum;
-    // fprintf(stdout, "%u ", sum);
+    _stop_optimizer += sum;
+    //fprintf(stdout, "%u ", sum);
   }
   for (i = 0; i < n; i++) {
     sum = 0;
@@ -115,8 +115,8 @@ int main(int argc, char **argv) {
       if (*(din + j * n + i) != NO_PATH)
         sum += *(din + j * n + i);
     }
-    realRes += sum;
-    // fprintf(stdout, "%u ", sum);
+    _stop_optimizer += sum;
+    //fprintf(stdout, "%u ", sum);
   }
   fprintf(stderr, "time for transitive stressmark = %f seconds.\n",
           difftime(stopTime, startTime));
diff --git a/dis/Update/update.c b/dis/Update/update.c
index b7bf2b5..a41f3e1 100644
--- a/dis/Update/update.c
+++ b/dis/Update/update.c
@@ -134,10 +134,10 @@ int main(int argc, char **argv) {
   endTime = time(NULL);
 
   volatile int _stop_optimizer = hops;
-  // fprintf(stdout, "%u hops\n", hops);
+  //fprintf(stdout, "%u hops\n", hops);
   fprintf(stderr, "time for update stressmark = %f seconds.\n",
           difftime(endTime, startTime));
   free(field);
   WRITE_TO_FILE
-  return (1);
+  return (0);
 }
-- 
cgit v1.2.2