Skip to content

Fix some warnings#12918

Merged
snnn merged 3 commits into
mainfrom
snnn/p4
Sep 18, 2022
Merged

Fix some warnings#12918
snnn merged 3 commits into
mainfrom
snnn/p4

Conversation

@snnn
Copy link
Copy Markdown
Contributor

@snnn snnn commented Sep 10, 2022

Description:

Fix some warnings. For example:

member function but is not marked 'override' [-Werror,-Winconsistent-missing-override]
  virtual Status BeamSearch__Compute(const contrib::transformers::BeamSearch* p, OpKernelContext* ctx) { return p->contrib::transformers::BeamSearch::Compute(ctx); }

Motivation and Context

  • Why is this change required? What problem does it solve?
  • If it fixes an open issue, please link to the issue here.

@snnn snnn merged commit e02bea2 into main Sep 18, 2022
@snnn snnn deleted the snnn/p4 branch September 18, 2022 17:55
linnealovespie pushed a commit that referenced this pull request Sep 30, 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.

2 participants