[PATCH] D54356: Convert CheckICE into a statment visitor
Bill Wendling via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Nov 13 12:41:32 PST 2018
void added a comment.
This code is called in over 90 places, so it's hard to tell if they all are in a constant context. Though I suppose that what this code is supposed to check for would work the same in a constant context as without one. I can revert this if you want, but to be honest the original function was terrible--it's huge and hard to understand what's going on. As for adding new expressions, this isn't the only place where a `StmtVisitor` is used. One still needs to go through all of those visitors to see if they need to add their expression.
Repository:
rC Clang
https://reviews.llvm.org/D54356
More information about the cfe-commits
mailing list