Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Commit

Permalink
fill reqs vector with num of outputs
Browse files Browse the repository at this point in the history
  • Loading branch information
azai91 committed Jun 15, 2018
1 parent 47e9983 commit 60e1d0b
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions tests/cpp/operator/mkldnn.cc
Original file line number Diff line number Diff line change
Expand Up @@ -708,7 +708,7 @@ TEST(MKLDNN_NDArray, CopyFrom) {
void TestOp(const OpAttrs &attrs, InitFunc init_fn, VerifyFunc verify_fn) {
std::vector<NDArray*> inputs(attrs.num_inputs);
std::vector<NDArray*> outputs(attrs.num_outputs);
std::vector<OpReqType> req(1);
std::vector<OpReqType> req(attrs.num_inputs);
std::vector<DispatchMode> dispatches = attrs.dispatches;

TestArrayShapes tas = GetTestArrayShapes();
Expand All @@ -719,9 +719,11 @@ void TestOp(const OpAttrs &attrs, InitFunc init_fn, VerifyFunc verify_fn) {
for (auto dispatch : dispatches) {
std::vector<NDArrayAttrs> out_arrs = GetTestOutputArrays(in_arr.arr.shape(), pds, init_fn);
for (auto out_arr : out_arrs) {
req[0] = kWriteTo;
for (int i = 0; i < attrs.num_inputs; i++)
inputs[i] = &in_arr.arr;
for (int i = 0; i < attrs.num_outputs; i++)
req[i] = kWriteTo;

outputs[0] = &out_arr.arr;
PrintVerifyMsg(in_arr, out_arr);
Imperative::Get()->InvokeOp(Context(), attrs.attrs, inputs,
Expand All @@ -738,11 +740,11 @@ void TestOp(const OpAttrs &attrs, InitFunc init_fn, VerifyFunc verify_fn) {
// If the array is a view, we shouldn't write data to it.
if (arr.arr.IsView())
continue;

NDArrayAttrs orig(arr.arr.Copy(arr.arr.ctx()), "InPlace Copy");
req[0] = kWriteInplace;
for (int i = 0; i < attrs.num_inputs; i++)
inputs[i] = &arr.arr;
for (int i = 0; i < attrs.num_outputs; i++)
req[i] = kWriteInplace;
outputs[0] = &arr.arr;
PrintVerifyMsg(orig, arr);
Imperative::Get()->InvokeOp(Context(), attrs.attrs, inputs, outputs, req,
Expand Down

0 comments on commit 60e1d0b

Please sign in to comment.