andrewrk marked an inline comment as done. andrewrk added a comment. In https://reviews.llvm.org/D51265#1215060, @efriedma wrote:
> What is this change actually solving? Even if the typedef is actually > redundant, it shouldn't cause a compiler error: redundant typedefs are > allowed in the latest versions of the C and C++ standards. (And even if you > were targeting an earlier standard, we normally suppress warnings in system > headers.) Here's an example use case that this solves: - Install MXE (See https://mxe.cc/) - Install the png package - Use libclang API to parse png.h - Expect that there are no errors, but there are errors Repository: rC Clang https://reviews.llvm.org/D51265 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits