[clang] [Clang] Fix null pointer dereference in enum debug info generation (PR #97105)

via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 23 15:18:43 PDT 2024


smanna12 wrote:

> Thanks @smanna12. I think this looks ok; returning null here does appear to be consistent with other overloads of `CreateTypeDefinition` for `RecordType` and `ObjCInterfaceType`. I agree with @Michael137 that it would be nice to have an example that fails the added condition. The code already checks for an incomplete type so is presumably intended to handle such types. Perhaps we are missing a test though? Presumably one that uses a forward declareable enum type?

I have added test. I could not able to figure out about how we can cover the scenario where getDefinition() returns nullptr. 

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


More information about the cfe-commits mailing list