Skip to content

Conversation

@baoachun
Copy link
Contributor

PR types

Others

PR changes

OPs

Describe

1、添加CPU的matmul_v2映射v1的pass。
2、修复当开启reshape_transpose_matmul_mkldnn_fuse_pass之后,reshape算子的reshape属性中有-1时,mamtul和matmul_mkldnn对输入维度计算错误问题。

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

if (it_zero != shape.end()) {
for (uint64_t i = 0; i < shape.size(); i++) {
if (shape[i] == 0) {
PADDLE_ENFORCE_LE(i, dim.size(),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LT? i 不能等于 dim.size()

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修改

if (it_zero != shape.end()) {
for (uint64_t i = 0; i < shape.size(); i++) {
if (shape[i] == 0) {
PADDLE_ENFORCE_LE(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

同上

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已修改

@cryoco cryoco merged commit d5245a3 into PaddlePaddle:develop Oct 27, 2021
@baoachun baoachun deleted the matmul_v2_to_v1 branch October 29, 2021 09:16
ghost pushed a commit to piotrekobi/Paddle that referenced this pull request Nov 3, 2021
…6731) * fix matmul dim error * fix wrong dim check in matmul
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants