[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