remove event methods from context for unit tests. Also a format change to suit the odd taste

src/opt/dom_info.cpp

@@ -156,9 +156,7 @@ bool dom_info::is_split_point(instruction_ref ins)
         if(arg_stream >= 0)
             stream_set.insert(arg_stream);
     }
-    if(stream_set.size() > 1)
-        return true;
-    return false;
+    return (stream_set.size() > 1) ? true : false;
 }
 
 // Identify merge points.  A merge point has more than one
@@ -172,9 +170,7 @@ bool dom_info::is_merge_point(instruction_ref ins)
         if(arg_stream >= 0)
             stream_set.insert(arg_stream);
     }
-    if(stream_set.size() > 1)
-        return true;
-    return false;
+    return (stream_set.size() > 1) ? true : false;
 }
 
 //  Propagate split points through the graph and identify concurrent instructions.

test/const_eval_test.cpp

@@ -47,10 +47,6 @@ struct non_computable_cf
 struct test_context
 {
     void finish() const {}
-    void set_stream(int) {}
-    void create_events(int) {}
-    void record_event(int) {}
-    void wait_event(int) {}
 };
 
 TEST_CASE(literal_test)

test/eval_test.cpp

@@ -11,10 +11,6 @@ struct id_target
     struct context
     {
         void finish() const {}
-        void set_stream(int) {}
-        void create_events(int) {}
-        void record_event(int) {}
-        void wait_event(int) {}
     };
     migraphx::context ctx = context{};
     std::string name() const { return "id"; }