Skip to content

Remove useless headers for some grad ops #38732

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

JamesLim-sy
Copy link
Contributor

@JamesLim-sy JamesLim-sy commented Jan 5, 2022

PR types

Others

PR changes

OPs

Describe

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jan 5, 2022

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

@JamesLim-sy JamesLim-sy force-pushed the remove_useless_header_in_elementwise_mul_grad branch from 0bffe58 to f3c5934 Compare January 5, 2022 14:49
@JamesLim-sy JamesLim-sy changed the title Remove useless header sub\mul\div grad Remove useless header for some grad op Jan 5, 2022
@JamesLim-sy JamesLim-sy changed the title Remove useless header for some grad op Remove useless header for some grad ops Jan 5, 2022
@JamesLim-sy JamesLim-sy changed the title Remove useless header for some grad ops Remove useless headers for some grad ops Jan 6, 2022
@@ -13,6 +13,7 @@ See the License for the specific language governing permissions and
limitations under the License. */

#include "paddle/fluid/operators/elementwise/elementwise_div_op.h"
#include "paddle/fluid/operators/reduce_ops/reduce_op.cu.h"

Copy link
Contributor

Choose a reason for hiding this comment

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

#include "paddle/fluid/operators/reduce_ops/reduce_op.cu.h" 可以不需要加吧

#include "paddle/fluid/operators/reduce_ops/reduce_op.cu.h"
#include "paddle/fluid/platform/complex.h"
Copy link
Contributor

Choose a reason for hiding this comment

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

#include "paddle/fluid/operators/reduce_ops/reduce_op.cu.h" 可以不需要加

Copy link
Contributor Author

@JamesLim-sy JamesLim-sy Jan 6, 2022

Choose a reason for hiding this comment

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

Cool ! I wiil revert this pr for completely remove of total useless headers, thanks

Copy link
Contributor

@ZzSean ZzSean left a comment

Choose a reason for hiding this comment

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

LGTM

@JamesLim-sy JamesLim-sy merged commit c0e2b98 into PaddlePaddle:develop Jan 6, 2022
JamesLim-sy added a commit that referenced this pull request Jan 6, 2022
JamesLim-sy added a commit that referenced this pull request Jan 6, 2022
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