Skip to content

Conversation

@AlexandreSinger
Copy link
Contributor

The rhel9 build of VTR is unable to resolve the "onetbb" prefix in the include path for tbb. This is easily resolved by just removing this prefix.

Other tbb includes do not use this prefix. This is only found in this one place.

The rhel9 build of VTR is unable to resolve the "onetbb" prefix in the
include path for tbb. This is easily resolved by just removing this
prefix.
@github-actions github-actions bot added VPR VPR FPGA Placement & Routing Tool lang-cpp C/C++ code labels Jul 3, 2025
@AmirhosseinPoolad
Copy link
Contributor

Oh hey my quest to use Fedora might actually prove useful to someone! Kind of surprising that it didn't work on RHEL9 but also I've only used Fedora 40 and 41 and RHEL 9 is AFAIK based on Fedora 34.

LGTM.

@AlexandreSinger
Copy link
Contributor Author

Oh hey my quest to use Fedora might actually prove useful to someone! Kind of surprising that it didn't work on RHEL9 but also I've only used Fedora 40 and 41 and RHEL 9 is AFAIK based on Fedora 34.

LGTM.

Well, you certainly made some random users of Silicon Compiler happy lol: siliconcompiler/siliconcompiler#3764 (comment)

SC maintains support for Ubuntu 20, 22, and 24 as well as Rhel.8 and 9. We dropped the support for Ubuntu 20 and Rhel 8; but with this we can keep the support for Rhel 9 unless I find any more issues.

I was suprised that this broke since you use Fedora and do not hit this issue and all the Ubuntu builds do not have an issue either. But whatever, easy fix.

@AlexandreSinger AlexandreSinger merged commit 4bb340a into verilog-to-routing:master Jul 3, 2025
33 checks passed
@AlexandreSinger AlexandreSinger deleted the feature-fix-tbb-include branch July 3, 2025 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang-cpp C/C++ code VPR VPR FPGA Placement & Routing Tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants