Skip to content

[onert/cpu] Fix exception throw for unsupported data type#16428

Merged
hseok-oh merged 1 commit intoSamsung:masterfrom
arkq:one-cpu-typo-fix
Mar 10, 2026
Merged

[onert/cpu] Fix exception throw for unsupported data type#16428
hseok-oh merged 1 commit intoSamsung:masterfrom
arkq:one-cpu-typo-fix

Conversation

@arkq
Copy link
Contributor

@arkq arkq commented Mar 9, 2026

This commit fixes a typo which results in the exception not being thrown.

Copilot AI review requested due to automatic review settings March 9, 2026 15:36
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bug in ResizeBilinearLayer::run() where std::runtime_error was constructed without the throw keyword, meaning the exception was silently discarded and execution would fall through or continue incorrectly when encountering unsupported data types.

Changes:

  • Added the missing throw keyword to the std::runtime_error in the default case of the data type switch.
  • Consolidated the previously separate "NYI" (not yet implemented) cases (INT32, INT64, QUANT_INT8_SYMM) to fall through to the default case.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

This commit fixes a typo which results in the exception not being
thrown.

ONE-DCO-1.0-Signed-off-by: Arkadiusz Bokowy <a.bokowy@samsung.com>
@arkq arkq force-pushed the one-cpu-typo-fix branch from 3ee507d to 5e89204 Compare March 9, 2026 17:24
Copy link
Contributor

@hseok-oh hseok-oh left a comment

Choose a reason for hiding this comment

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

LGTM

@hseok-oh hseok-oh merged commit 6a868a9 into Samsung:master Mar 10, 2026
9 checks passed
@arkq arkq deleted the one-cpu-typo-fix branch March 10, 2026 05:32
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