Skip to content
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

[HW] InnerSymProperties: emit error when visibility is invalid #5769

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

youngar
Copy link
Member

@youngar youngar commented Aug 3, 2023

There is no parseKeyword which takes a list of alternatives, so we have to emit the error ourselves. This could be fixed with some better upstream functionality.

Fixes #5597

@youngar youngar added the HW Involving the `hw` dialect label Aug 3, 2023
@youngar youngar requested a review from dtzSiFive August 3, 2023 04:12
There is no `AsmParser::parseKeyword` which takes a list of alternatives, so we
have to emit the error ourselves. This could be fixed with some better upstream
functionality.

Fixes llvm#5597
@youngar youngar merged commit 5af0427 into llvm:main Aug 3, 2023
5 checks passed
@youngar youngar deleted the hw-innersymprop-parsing branch August 3, 2023 06:05
@dtzSiFive
Copy link
Contributor

My hero, ty!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HW Involving the `hw` dialect
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIRRTL] Silent parsing failure with invalid inner symbol visibility specifier
3 participants