Skip to content

Refine platform::Transform function and fix prelu_op testing.#4201

Merged
qingqing01 merged 3 commits intoPaddlePaddle:developfrom
qingqing01:fix_prelu
Sep 20, 2017
Merged

Refine platform::Transform function and fix prelu_op testing.#4201
qingqing01 merged 3 commits intoPaddlePaddle:developfrom
qingqing01:fix_prelu

Conversation

@qingqing01
Copy link
Contributor

@qingqing01 qingqing01 commented Sep 19, 2017

Fix #4202
Fix #4204

Thanks for @hedaoyuan's help.

};

#ifdef __NVCC__
template <>
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

@qingqing01 qingqing01 Sep 20, 2017

Choose a reason for hiding this comment

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

@jacquesqiao 改了之后,带了模板参数,对于CPUPlace 、 GPUPlace编译出来是两个不同的符号。

原先那样,在.cu文件编译后,链接时,用的是第一次g++/gcc编译同出来的同名符号,就出现问题。

Copy link
Collaborator

@reyoung reyoung left a comment

Choose a reason for hiding this comment

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

I think this #4209 is much simpler.

@hedaoyuan
Copy link
Contributor

Such as the comment in the pr 4209, this code is more efficient after compile.

@qingqing01 qingqing01 merged commit d865b04 into PaddlePaddle:develop Sep 20, 2017
@qingqing01 qingqing01 deleted the fix_prelu branch November 14, 2019 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants