Skip to content

Eliminate the assert in specific situations.#156

Merged
giulianobelinassi merged 1 commit intoSUSE:mainfrom
earnol:main
Oct 10, 2025
Merged

Eliminate the assert in specific situations.#156
giulianobelinassi merged 1 commit intoSUSE:mainfrom
earnol:main

Conversation

@earnol
Copy link

@earnol earnol commented Oct 10, 2025

The llvm code generates an assert originating from https://clang.llvm.org/doxygen/DeclBase_8cpp_source.html#l01047 in case of requesting attributes when attributes are not there. This change prevents assert in this situation.

@earnol earnol changed the title Eliminate the assert in cirtain situations. Eliminate the assert in specific situations. Oct 10, 2025
The llvm code generates an assert originating from
https://clang.llvm.org/doxygen/DeclBase_8cpp_source.html#l01047 in
case of requesting attributes when attributes are not there.
This change addresses this case.
@giulianobelinassi
Copy link
Collaborator

Thanks. Perhaps some LLVM versions ago it always gave an empty vector so it wasn't a problem?

@giulianobelinassi giulianobelinassi merged commit 1fcfc0a into SUSE:main Oct 10, 2025
8 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.

2 participants