dwavesystems / dwavesystems/dwave-optimization
Consider switching to LLVM-style RTTI for `Node`s and `NodeStateData`
Open
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
Assessment
This issue has not been assessed yet.