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

Reply via email to