dwavesystems / dwavesystems/dwave-optimization

Consider switching to LLVM-style RTTI for `Node`s and `NodeStateData`

Open
#372 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
31
Forks
36
Avg merge
16h 55m
Merged PRs (30d)
8

Description

See https://llvm.org/docs/HowToSetUpLLVMStyleRTTI.html

The upshot is keeping an enum of all possible node types and then storing that value on each instantiated object. The benefit is much faster type checking and casting, the downside is a small increase in the memory footprint of each instantiated object. Also a decent amount of boilerplate.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.