[PATCH] D137415: [clang][Interp] Implement switch statements

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 21 18:28:59 PST 2022


shafik added inline comments.


================
Comment at: clang/lib/AST/Interp/ByteCodeStmtGen.cpp:405
+  const Expr *Cond = S->getCond();
+  PrimType CondT = this->classifyPrim(Cond->getType());
+
----------------
The condition could be a class type w/ a conversion operator, this does not seem like it will handle this case. Maybe b/c you don't handle conversion operators yet but might be worth a `FIXME` or `TODO`.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137415/new/

https://reviews.llvm.org/D137415



More information about the cfe-commits mailing list