[PATCH] D54356: Convert CheckICE into a statment visitor

[Bill Wendling via Phabricator via cfe-commits]

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