-
Notifications
You must be signed in to change notification settings - Fork 463
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Missing namespaces in tofino back end. #5214
Comments
@fruffy taking on this task. anything else you would like to tell |
This is a fairly straightforward task but you will have to edit many files. You may also run into compilation problems because of the namespace change, which may need to be fix. Usually the change is simple, but it can be tedious. Another thing: Anything like |
ok i will start working on this |
Why? bfas is a standalone executable, not a library, so there's no point to using namespaces other than obfuscation. |
Similar reasoning as #4707 (comment). If you want to use the assembler as library this could become an issue (although it's unlikely that will be the case for now). bfas is also not completely standalone, it does call into |
A lot of the files in the
tofino
backend havenamespace BFN
but it should benamespace P4:BFN
. A lot of the files (in particular in bf-asm) do not have any namespace at all. This should be fixed.The text was updated successfully, but these errors were encountered: