[clang] [analyzer] Fix crash in PaddingChecker on dependent field types (PR #232986)
Manuel Sainz de Baranda y Goñi via cfe-commits
cfe-commits at lists.llvm.org
Sat Oct 17 12:06:28 PDT 2026
redcode wrote:
> Unless there is urgency - I'd prefer a Sema fix over a workaround. I can also understand that crashing is not really a good outcome while waiting on the proper fix - so this is why I'm asking. Would it work for you all to wait some?
TBH, I don't think the fix does any harm. I'm willing to investigate this further and work on a separate PR where we'd fix Sema, add the unit test you asked for, and add an assert to the guard in `PaddingChecker::shouldSkipDecl` so we don't mask potential future bugs.
But you have more experience than me, so if you think it's not a good idea, I'll close the PR.
https://github.com/llvm/llvm-project/pull/232986
More information about the cfe-commits
mailing list