[llvm-branch-commits] [clang] release/18.x: [Clang] Handle structs with inner structs and no fields (#89126) (PR #89456)

Eli Friedman via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Fri Apr 19 14:41:07 PDT 2024


efriedma-quic wrote:

LGTM

This only impacts code using dynamic object sizes, which... I'm not sure how widely it's actually used outside the Linux kernel.  Implementation-wise, should be pretty safe.  There's some minor risk because the revised recursion visits RecordDecls it wouldn't look into before, but that seems unlikely to cause a practical issue.

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


More information about the llvm-branch-commits mailing list