Skip to content

Conversation

@leo0519
Copy link
Collaborator

@leo0519 leo0519 commented Dec 23, 2021

PR types

Function optimization

PR changes

OPs

Describe

Replace GeLU plugin with TensorRt built-in layer after TensorRT 7.0.
The benefit is that TensorRT can fuse the fc + gelu layer.
We implement both approximate and nonapproximate GeLU.

@paddle-bot-old
Copy link

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

@jeng1220
Copy link
Collaborator

@zhouping828 ,
Please help review. Thanks

Copy link

@zhouping828 zhouping828 left a comment

Choose a reason for hiding this comment

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

LGTM

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jan 3, 2022

Sorry to inform you that 63ea3f1's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@leo0519 leo0519 added the NVIDIA label Jan 6, 2022
Copy link
Contributor

@Wangzheee Wangzheee left a comment

Choose a reason for hiding this comment

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

LGTM

@Wangzheee Wangzheee merged commit a3689d8 into PaddlePaddle:develop Feb 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4 participants