Skip to content

Conversation

Slarkarus
Copy link
Contributor

PR for #1919

@xlauko
Copy link
Collaborator

xlauko commented Sep 29, 2025

This does not fix #1919 fully as it is issue in the incubator from where we generate docs.
Please fix it there too, thank you.

Copy link
Collaborator

@xlauko xlauko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Slarkarus
Copy link
Contributor Author

Slarkarus commented Oct 1, 2025

This does not fix #1919 fully as it is issue in the incubator from where we generate docs. Please fix it there too, thank you.

Sorry, i don't understand, isn't llvm/clangir an incubator? Where I need to fix this bug?

@bcardosolopes
Copy link
Member

Sorry, i don't understand, isn't llvm/clangir an incubator? Where I need to fix this bug?

I think he's just saying that this will fix the tablegen code, but something else has to run behind the scenes to update the docs since it's not automatic (I'll do that no worries).

@bcardosolopes bcardosolopes merged commit 55cf31c into llvm:main Oct 3, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants