Changeset 8f557161 for src/Common/Eval.h

Timestamp:

Jun 13, 2023, 4:33:16 PM (13 months ago)

Author:

Michael Brooks <mlbrooks@…>

Branches:

master

Children:

5668740, fec8bd1

Parents:

576aadb

Message:

Clarify and fix accuracy in eval public API, on reporting "unable to evaluate."

While the eval internals always used the flag pair valid and cfavalid almost correctly, the public interface exposed the outcome as a single flag, and the various interpretations of this flag were a mess.

Old cfacc treated sizeof(whatever) in some contexts as "known to be zero," which is wrong.
The generally correct answer, which new cfacc now uses is, "unknown now, but GCC will see it as a fine constant."
New tests/eval.cfa captures this fact: is runnable and would fail on old cfacc; it passes with new cfacc.

File:

• src/Common/Eval.h (modified) (1 diff)

src/Common/Eval.h

@@ -23 +23 @@
 }
 
+struct Evaluation {
+        long long int knownValue;
+        bool hasKnownValue;
+        bool isEvaluableInGCC;
+};
+
 /// Evaluates expr as a long long int.
 /// If second is false, expr could not be evaluated.
 std::pair<long long int, bool> eval(const Expression * expr);
-std::pair<long long int, bool> eval(const ast::Expr * expr);
+Evaluation eval(const ast::Expr * expr);
 
 // Local Variables: //