[clang] [Clang][Sema] Fix crash in __builtin_dump_struct with immediate callables (PR #192880)

Oliver Hunt via cfe-commits cfe-commits at lists.llvm.org
Mon May 4 11:55:10 PDT 2026


ojhunt wrote:

While the final fix looks good I think the initial PR violated the LLVM rules around the use of LLMs which require a human review that understands the change - not understanding the explanation produced by the LLM, but the change itself. A moments examination of the file directly would be enough to see that `return E` was the correct implementation, a position supported by the numerous equivalent methods in the class.

There were also problems with the comments: even when given the fix you used an LLM to generate a needlessly large comment rather than the explanation yourself.

https://github.com/llvm/llvm-project/pull/192880


More information about the cfe-commits mailing list