[clang] [clang] Move two flags from EvalInfo to State (PR #157046)

[via cfe-commits]

================
@@ -960,19 +947,6 @@ namespace {
       EM_IgnoreSideEffects,
     } EvalMode;
 
-    /// Are we checking whether the expression is a potential constant
-    /// expression?
-    bool checkingPotentialConstantExpression() const override  {
-      return CheckingPotentialConstantExpression;
-    }
-
-    /// Are we checking an expression for overflow?
-    // FIXME: We should check for any kind of undefined or suspicious behavior
-    // in such constructs, not just overflow.
----------------
Sirraide wrote:

Yeah, I haven’t look at how it’s used admittedly, but I understood that FIXME to mean ‘it’s called `checkingForUndefinedBehavior()` with the intent that it eventually should check for more than just overflow’.

https://github.com/llvm/llvm-project/pull/157046