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

Reply via email to