[flang-commits] [flang] [Flang][#212316] Fix -Wunused-template errors under -Werror (PR #218985)

[Aiden Grossman via flang-commits]

================
@@ -15,8 +15,7 @@ namespace Fortran::evaluate {
 
 // DOT_PRODUCT
 template <typename T>
-static Expr<T> FoldDotProduct(
-    FoldingContext &context, FunctionRef<T> &&funcRef) {
+Expr<T> FoldDotProduct(FoldingContext &context, FunctionRef<T> &&funcRef) {
----------------
boomanaiden154 wrote:

This is wrong. The C++ standard allows for multiple of the same template definition in different TUs.

If you're not going to be able to validate the output of an LLM appropriately, please do not post LLM generated comments.

https://github.com/llvm/llvm-project/pull/218985