[PATCH] D13344: Keep the IfStmt node even if the condition is invalid

Milian Wolff via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 1 07:27:07 PDT 2015


milianw added a comment.

Is there still an error reported for the invalid condition?

Anyhow, from my POV this is excellent and should fix a bunch of issues I've seen when using clang-c in KDevelop. I never got around to investigating it, but it always was related to conditionals. I'm pretty sure this patch fixes it then.

@ogoffart: For Manuel to see the difference, could you share the before/after screenshots of your code browser? That makes the impact of this patch pretty clear.


http://reviews.llvm.org/D13344





More information about the cfe-commits mailing list