[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