Skip to content

add config.SetMkldnnCacheCapacity api for mkldnn cache clear strategy#18579

Closed
luotao1 wants to merge 4 commits intoPaddlePaddle:developfrom
luotao1:enable_mkldnn_enhance_strategy
Closed

add config.SetMkldnnCacheCapacity api for mkldnn cache clear strategy#18579
luotao1 wants to merge 4 commits intoPaddlePaddle:developfrom
luotao1:enable_mkldnn_enhance_strategy

Conversation

@luotao1
Copy link
Contributor

@luotao1 luotao1 commented Jul 10, 2019

  1. add config.SetMkldnnCacheCapacity(xxx) api for mkldnn cache clear strategy, xxx means the mkldnn_cache_capacity for difference input shape.
  2. simplify the TEST(Analyzer_MM_DNN, mkldnn_cache_clear) with the new api, and add output compare between no cache strategy and using cache strategy.
  3. add the TEST(Analyzer_MM_DNN, mkldnn_shape_blob_size) using AnalysisPredictor::ZeroCopyRun, since the new api will reset the cur_mkldnn_session_id to default.

related #18549

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ luotao1
❌ guofei02


guofei02 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@luotao1 luotao1 requested a review from a user July 10, 2019 10:51
@luotao1
Copy link
Contributor Author

luotao1 commented Jul 10, 2019

@LeoZhao-Intel I move #18549 to here.

@luotao1 luotao1 closed this Jul 10, 2019
@luotao1 luotao1 reopened this Jul 10, 2019
@luotao1 luotao1 closed this Jul 10, 2019
@luotao1 luotao1 deleted the enable_mkldnn_enhance_strategy branch August 29, 2019 08:25
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