================
@@ -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
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
