-
Notifications
You must be signed in to change notification settings - Fork 449
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
Also include the backend IR files in the ir-generated library file. #3985
Conversation
There seems to be some non-determinism in the p4c-bm2 json serialization (or json loading?) for Ubuntu 18. Not quite sure what could cause that. Maybe it is using unordered maps? Edit: Protobuf maps do not necessarily preserve ordering. Could that be the reason? |
b1fab14
to
b9d6daa
Compare
I don't think the tests in load_ir_from_json.cpp have anything to do with Protobuf. That's IR serialization. |
Ah, for some reason I assumed the JSON generated by |
1509a17
to
4fda0d1
Compare
Turns out |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for fixing the build issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rubberstamp'd.
Thanks! |
Hopefully fixes #3984.