After D80369, the retainedTypes in CU's should not have any
subprograms so we should not handle that case when emitting debug info.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
I found this while looking through the callsites to getOrCreateSubprogramDIE(). I was thinking of adding an assert to check that retainedTypes don't include SPs but it seems there are a lot of tests that do this.
Comment Actions
This looks reasonable to me.
The call_site entries still point to the right subprograms after this patch, right?
retainedTypes: !3 - this should be removed completely, so we do not leave any ambiguity