pytorch
c5a67f16 - Fix minor inaccuracy in translate error reporting (#53032)

Commit
4 years ago
Fix minor inaccuracy in translate error reporting (#53032) Summary: Pull Request resolved: https://github.com/pytorch/pytorch/pull/53032 Previously, you could get this error message: ``` Failed to synthesize the expression "Tensor & out". When I failed, the following bindings were available in the context: const Tensor & self; const Tensor & other; Scalar alpha; const Tensor & op.outputs_[0]; ``` There's a problem with this error message: it doesn't seem like there is any 'out' argument available, but actually there is: the last binding in the context is it. We printed the *expression*, not the *ctype name*. After this patch, the context now prints as: ``` const Tensor & self; // self const Tensor & other; // other Scalar alpha; // alpha const Tensor & out; // op.outputs_[0] ``` Now it becomes clear that it's a const mismatch. Maybe we could also beef up the error message so it points out near misses, but I'll leave that to future work. Signed-off-by: Edward Z. Yang <ezyang@fb.com> Test Plan: Imported from OSS Reviewed By: ljk53 Differential Revision: D26729768 Pulled By: ezyang fbshipit-source-id: adb363551a7145eac788943c20969c86b1f8a81b
Author
Parents
Loading