incorporate code changes from the review feedback

src/include/migraphx/matcher.hpp

@@ -308,12 +308,6 @@ struct matcher_result
         {
         }
 
-        instruction_container& operator=(const std::unordered_map<std::string, instruction_ref>& x)
-        {
-            ins_map = x;
-            return *this;
-        }
-
         instruction_ref operator[](const std::string& name) const
         {
             auto it = ins_map.find(name);

test/verify/run_verify.cpp

@@ -44,8 +44,8 @@ MIGRAPHX_DECLARE_ENV_VAR(MIGRAPHX_DUMP_TEST)
 
 // An improved async, that doesn't block
 template <class Function>
-std::future<typename std::result_of<Function()>::type> detach_async(Function&& f,
-                                                                    bool parallel = true)
+std::future<typename std::invoke_result_t<Function>> detach_async(Function&& f,
+                                                                  bool parallel = true)
 {
     if(parallel)
     {