erichkeane wrote: > I've noticed that this PR changed the tag type kind of the implicit > `type_info` from `class` to `struct`. I don't know if it has any bad > consequences, but it looks inconsistent with the definition of > `std::type_info`. > > Seems like it is a `class` in MSVC: https://godbolt.org/z/zj9asdE1a
Oh, interesting! class vs struct can mangle differently in certain ABIs (and IIRC, Windows is one that it does!). That said, we mangle type_info special, so there is no real way I could find to notice the difference between class-vs-struct here. Either way, fixed here: 16766b3aef281a24acad08292ffe69554b44805b https://github.com/llvm/llvm-project/pull/151277 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits