Merge pull request #156 from mingzhe09088/fix-scalar-brace

prigoyal · web-flow · commit 8c94829678e5 · 2018-03-14T15:01:28.000-04:00
Fix compilation issue: braces around scalar initializer
diff --git a/include/tc/c2/copy_op.h b/include/tc/c2/copy_op.h
@@ -47,7 +47,7 @@ class TcCopyOp : public TcOp<T, Context, Engine> {
                                 .tile({4, 8, 8})
                                 .mapToThreads({32, 4, 4})
                                 .mapToBlocks({100, 100, 100})
-                                .unroll({128});
+                                .unroll(128);
     this->gradMappingOptions_ =
         tc::MappingOptions::makePointwiseMappingOptions();
   }
diff --git a/test/test_basic.cc b/test/test_basic.cc
@@ -87,7 +87,7 @@ TEST(Math, Median) {
   std::vector<int> v0{};
   EXPECT_THROW(tc::median(v0), std::out_of_range);
 
-  std::vector<int> v1{{1}};
+  std::vector<int> v1({1});
   EXPECT_EQ(tc::median(v1), 1);
 
   std::vector<int> v2{{1, 3}};

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ class TcCopyOp : public TcOp<T, Context, Engine> {`
`47`	`47`	`.tile({4, 8, 8})`
`48`	`48`	`.mapToThreads({32, 4, 4})`
`49`	`49`	`.mapToBlocks({100, 100, 100})`
`50`		`- .unroll({128});`
	`50`	`+ .unroll(128);`
`51`	`51`	`this->gradMappingOptions_ =`
`52`	`52`	`tc::MappingOptions::makePointwiseMappingOptions();`
`53`	`53`	`}`