dblaikie added a comment. In D94735#2547370 <https://reviews.llvm.org/D94735#2547370>, @jdoerfert wrote:
> In D94735#2546968 <https://reviews.llvm.org/D94735#2546968>, @dblaikie wrote: > >> This patch is about a clang synthesized `struct`. >> `clang/lib/CodeGen/CGOpenMPRuntime.cpp:3463` has a synthesized `union` >> (`distinct !DICompositeType(tag: DW_TAG_union_type, name: "kmp_cmplrdata_t", >> size: 64, elements: <0x62b690>)`, which has no filename/line with this >> patch). It does not make sense to attach the current filename/line (which >> can be entirely irrelevant) to the synthesized `union`. @ABataev @jdoerfert ? > > There is no meaningful location for this generated type. Unclear if we should > use a filename/line to specify that, e.g. "clang_synthesized:0", or allow it > without. Either works for me. I'd vote in favour of having no location - lots of other types have no location (mostly builtin/fundamental types, admittedly) so should generally be fine. *fingers crossed* Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D94735/new/ https://reviews.llvm.org/D94735 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits