Fix building `rmp_abc_library` when `USE_SYSTEM_ABC` is set
(Small change)
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
I'm a bit confused by this. USE_SYSTEM_ABC is not mentioned anywhere in the PR beyond the title. What issue is being solved?
When USE_SYSTEM_ABC is set to true, ABC_LIBRARY is set to a non-null value. Things that require abc link against ABC_LIBRARY so ABC symbols won't be missing for those who set USE_SYSTEM_ABC to true.
rmp_abc_library was just missing ABC_LIBRARY.
rmp already links against it https://github.com/The-OpenROAD-Project/OpenROAD/blob/f0cc0ee1a62d7a6c35b43bd46a20980969eeb9a2/src/rmp/src/CMakeLists.txt#L73 - why does rmp_abc_library also need it (or should it just be moved)?
Because otherwise rmp_abc_library which is I believe compiled as a standalone unit will complain about being unable to find things in the abc:: namespace.