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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants