detect External indices more generally in internalize_constant (#2395)

diff --git a/build/instructions_template.rs b/build/instructions_template.rs
index 7914964f4deb7c618673db128f119bc0b89b924a..86cde2f36e461463e798cb806bbdf6b794afe1f3 100644 (file)
--- a/build/instructions_template.rs
+++ b/build/instructions_template.rs
@@ -1017,6 +1017,13 @@ fn generate_instruction_preface() -> TokenStream {
                     },
                 }
             }
+
+            pub fn is_external(&self) -> bool {
+                matches!(
+                    self,
+                    IndexingCodePtr::External(_) | IndexingCodePtr::DynamicExternal(_)
+                )
+            }
         }
 
         impl IndexingInstruction {

diff --git a/src/indexing.rs b/src/indexing.rs
index 4eb39e2de50726c8afadd02cc354882ceff9408d..2df0553086b37d7e2337397c06697f89278c597f 100644 (file)
--- a/src/indexing.rs
+++ b/src/indexing.rs
@@ -114,13 +114,13 @@ impl<'a> IndexingCodeMergingPtr<'a> {
             Some(OptArgIndexKey::Literal(_, _, constant, _)) => {
                 constants.insert(*constant, constant_ptr);
             }
+            _ if constant_ptr.is_external() => {
+                // this must be a defunct clause, because it's been deleted
+                // from the skeleton.
+                debug_assert!(constant_key.is_none());
+            }
             _ => {
-                if let IndexingCodePtr::DynamicExternal(_) = constant_ptr {
-                    // this must be a defunct clause, because it's been deleted
-                    // from the skeleton.
-                } else {
-                    unreachable!()
-                }
+                unreachable!()
             }
         }