================
@@ -344,10 +352,17 @@ class ComplexExprEmitter
     return QualType();
   }
 
+  template <typename T>
+  FPOptionsOverride getStoredFPFeaturesOrDefault(const T *E,
+                                                 const CodeGenFunction &CGF) {
----------------
AaronBallman wrote:

`CGF` is unused, can it be removed?

Why a template type? Everything is passing in some subclass of `Expr` it seems.

Also, the function should be marked `static`, but why not add it as a member 
function alongside `getStoredFPFeatures()`?

https://github.com/llvm/llvm-project/pull/98520
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to