Skip to content
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.

Sign up for GitHub

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

Jump to bottom

cinn_launch_op: switch to execution by PE #39911

Merged
merged 40 commits into from
Mar 3, 2022
Merged

cinn_launch_op: switch to execution by PE #39911

merged 40 commits into from
Mar 3, 2022

Conversation

CtfGo
Copy link
Contributor

@CtfGo CtfGo commented Feb 24, 2022

PR types

Performance optimization

PR changes

Others

Describe

  1. cinn_launch_op:upgrade the way of execution of the compiled instructions, currently it just launch the Execute inteface of the CINN runtime Program, after this PR, it will switch the new way that uses PE to schedule nodes of cinn_instruction_op
  2. fix several bug found in the progress of integration test.

CtfGo added 30 commits February 9, 2022 12:44
@CtfGo
add cinn_instruction_run_op for launching execution of a cinn instruc…
f7e9811 
…tion
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
81c7ebe 
… add_cinn_run_op
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
cc4e106 
… add_cinn_run_op
@CtfGo
fix multi definition compilation error
a645c3e
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
f64f8c2 
… add_cinn_run_op
@CtfGo
update cmake
b6fff8d
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
3220d21 
… add_cinn_run_op
@CtfGo
fix bug at infershape
79c0bb7
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
370cfb0 
… add_cinn_run_op
@CtfGo
fix compile error due to lacking header file
3937d3c
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
2462113 
… add_cinn_run_op
@CtfGo
add cinn_instruction_run_op test code
238fed4
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
f84c576 
… cinn_instruction_run_op_test
@CtfGo
update several interfaces of CinnLaunchContext
2fea564
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
88fcc60 
… cinn_instruction_run_op_test
@CtfGo
update several interfaces and add detail comments in CinnLaunchContex…
f5c99a2 
…t class
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
fe0556e 
… cinn_instruction_run_op_test
@CtfGo
to skip the bug of error message check
5a2f5f4
@CtfGo
to skip the bug of error message check
af3f3fe
@CtfGo
save the name lists of the input,internal and output variables of a s…
5c6f9dd 
…ubgraph as its attribute
@CtfGo
Merge branch 'cinn_instruction_run_op_test' of https://github.com/Ctf…
954c5fa 
…Go/Paddle into convert_cinn_program_to_graph
@CtfGo
Merge branch 'save_variable_lists_in_subgraph' of https://github.com/…
002a4c4 
…CtfGo/Paddle into convert_cinn_program_to_graph
@CtfGo
build a Paddle Graph from CINN compiled program for execution with PE
a84453e
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
44ba5ae 
… convert_cinn_program_to_graph
@CtfGo
update names of some variables
e64942b
@CtfGo
fix random fail in build_cinn_pass_test and update some comments
362901e
@CtfGo
Merge branch 'develop' of https://github.com/PaddlePaddle/Paddle into…
2beafaf 
… convert_cinn_program_to_graph
@CtfGo
fix compiler error by merging phi pr
7dda357
@CtfGo
swith to PE execution in cinn launch
a9d9ebc
@CtfGo
fix outer variables erased
43810ca
@paddle-bot-old
Copy link

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

zhhsplendid
zhhsplendid approved these changes Mar 3, 2022
View reviewed changes
Copy link
Member

@zhhsplendid zhhsplendid left a comment

Choose a reason for hiding this comment

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

LGTM

thisjiang
thisjiang approved these changes Mar 3, 2022
View reviewed changes
Copy link
Contributor

@thisjiang thisjiang left a comment

Choose a reason for hiding this comment

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

LGTM

@CtfGo CtfGo merged commit 167d511 into PaddlePaddle:develop Mar 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zhhsplendid zhhsplendid zhhsplendid approved these changes

@thisjiang thisjiang thisjiang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@CtfGo @zhhsplendid @thisjiang