Skip to content
This repository was archived by the owner on Apr 19, 2026. It is now read-only.
This repository was archived by the owner on Apr 19, 2026. It is now read-only.

"switch (dt)" does not include case "PLAIDML_DATA_BFLOAT16" in plaidml_translate.cpp #447

Description

@dbonner

ngraph-bridge fails to build with the plaidml backend, giving an error when compiling:
ngraph-bridge/build_cmake/ngraph/src/ngraph/runtime/plaidml/plaidml_translate.cpp

ngraph-bridge/build_cmake/ngraph/src/ngraph/runtime/plaidml/plaidml_translate.cpp:105:13: warning: 
      enumeration value 'PLAIDML_DATA_BFLOAT16' not handled in switch [-Wswitch]
    switch (dt)
            ^
ngraph-bridge/build_cmake/ngraph/src/ngraph/runtime/plaidml/plaidml_translate.cpp:105:13: note: 
      add missing switch cases
    switch (dt)
            ^
ngraph-bridge/build_cmake/ngraph/src/ngraph/runtime/plaidml/plaidml_translate.cpp:124:1: error: 
      control may reach end of non-void function [-Werror,-Wreturn-type]

I edited ngraph/runtime/plaidml/plaidml_translate.cpp and added under switch (dt):
case PLAIDML_DATA_BFLOAT16: return "as_bfloat16(" + tensor_name + ", 16)";

It then built successfully.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions