[PATCH] D19105: Changes in clang after running http://reviews.llvm.org/D18821

Alexander Kornienko via cfe-commits cfe-commits at lists.llvm.org
Wed May 11 02:16:16 PDT 2016


alexfh requested changes to this revision.
alexfh added a comment.
This revision now requires changes to proceed.

The bottom line is that the fixit in its current form is not particularly useful in most cases. We should try to detect some cases where the replacement is unlikely to be useful, e.g. the `enum { FollowRAUW = false };` pattern, returning a bool from a function that is declared to return a typedef to an integral type that contains `bool` in its name (e.g. `LLVMBool`), and maybe some other cases.


http://reviews.llvm.org/D19105





More information about the cfe-commits mailing list