disable deoptless in toplevel promise for now

reactorlabs · May 19, 2022 · f750863 · f750863
1 parent 2197f32
commit f750863
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/rir/src/compiler/native/builtins.cpp b/rir/src/compiler/native/builtins.cpp
@@ -854,8 +854,9 @@ SEXP deopt(rir::Code* c, SEXP cls, DeoptMetadata* m, R_bcstack_t* args,
     auto le = LazyEnvironment::check(env);
     if (deoptless && m->numFrames == 1 && cls != deoptlessRecursion &&
         ((le && !le->materialized()) ||
-         (!le && (!leakedEnv || !deoptlessNoLeakedEnvs)))) {
-        assert(m->frames[0].inPromise == false);
+         (!le && (!leakedEnv || !deoptlessNoLeakedEnvs))) &&
+        /* TODO: support deoptless when outermost frame is a promise */
+        !m->frames[0].inPromise) {
 
         size_t envSize = le ? le->nargs : Rf_length(FRAME(env));
         if (envSize <= DeoptContext::MAX_ENV &&