From 9dfc94d04d8f409d0e5bcdf40001d1b36dcbdc52 Mon Sep 17 00:00:00 2001 From: Doug Shi-Dong Date: Tue, 11 Aug 2020 22:50:59 -0400 Subject: [PATCH] Inform memory leak of sacado_rad_fad in doc. --- doc/doxygen/headers/automatic_and_symbolic_differentiation.h | 2 +- include/deal.II/differentiation/ad/ad_number_types.h | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/doc/doxygen/headers/automatic_and_symbolic_differentiation.h b/doc/doxygen/headers/automatic_and_symbolic_differentiation.h index 5d7e1394b9..3b3161cf5b 100644 --- a/doc/doxygen/headers/automatic_and_symbolic_differentiation.h +++ b/doc/doxygen/headers/automatic_and_symbolic_differentiation.h @@ -300,7 +300,7 @@ * - Forward-mode Sacado with dynamic memory allocation using expression templates (once differentiable) * - Nested forward-mode Sacado using expression templates (twice differentiable) * - Reverse-mode Sacado (once differentiable) - * - Nested reverse and dynamically-allocated forward-mode Sacado (twice differentiable) + * - Nested reverse and dynamically-allocated forward-mode Sacado (twice differentiable, but results memory leak described in Differentiation::AD::NumberTypes) * * Note that in the above, "dynamic memory allocation" refers to the fact that the number of * independent variables need not be specified at compile time. diff --git a/include/deal.II/differentiation/ad/ad_number_types.h b/include/deal.II/differentiation/ad/ad_number_types.h index 6bd7ae5716..c811d2699a 100644 --- a/include/deal.II/differentiation/ad/ad_number_types.h +++ b/include/deal.II/differentiation/ad/ad_number_types.h @@ -84,6 +84,11 @@ namespace Differentiation * * First derivatives will be computed using reverse mode, while the second * derivatives will be computed using forward mode. + * + * Note that the repeated use of the nested reverse-forward mode results + * in a memory leak described in this Trilinos + * issue. */ sacado_rad_dfad }; -- 2.39.5