Prevent taking a reference to a nullptr in frontend (#29048)

jabraham17 · web-flow · commit 036021c01eb1 · 2026-06-30T12:24:57.000-07:00
Adjusts some frontend compiler code to avoid potential nullptr dereferencing and subsequent taking a reference to a nullptr. [Reviewed by @benharsh]
diff --git a/frontend/include/chpl/resolution/ResolutionContext.h b/frontend/include/chpl/resolution/ResolutionContext.h
@@ -269,7 +269,8 @@ class ResolutionContext {
   }
   template <typename T>
   bool canUseGlobalCacheConsidering(const T* t) const {
-    return canUseGlobalCacheConsidering(*t);
+    if (!isUnstable()) return true;
+    return ResolutionContext::canUseGlobalCache(context_, t);
   }
   template <typename... Ts>
   bool canUseGlobalCacheConsidering(const std::tuple<Ts...>& ts) {

Original file line number	Diff line number	Diff line change
`@@ -269,7 +269,8 @@ class ResolutionContext {`
`269`	`269`	`}`
`270`	`270`	`template <typename T>`
`271`	`271`	`bool canUseGlobalCacheConsidering(const T* t) const {`
`272`		`- return canUseGlobalCacheConsidering(*t);`
	`272`	`+ if (!isUnstable()) return true;`
	`273`	`+ return ResolutionContext::canUseGlobalCache(context_, t);`
`273`	`274`	`}`
`274`	`275`	`template <typename... Ts>`
`275`	`276`	`bool canUseGlobalCacheConsidering(const std::tuple<Ts...>& ts) {`