From f6c558b41d3e003e1626a853aff0b58705847e84 Mon Sep 17 00:00:00 2001
From: ledwards2225 <98505400+ledwards2225@users.noreply.github.com>
Date: Thu, 29 Feb 2024 03:38:12 -0700
Subject: [PATCH] fix: add TODO with issue for num_gates bug (#4847)

See issue https://github.com/AztecProtocol/barretenberg/issues/875
---
 .../proof_system/circuit_builder/ultra_circuit_builder.hpp     | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.hpp b/barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.hpp
index fef275152d2..00f6ca265e7 100644
--- a/barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.hpp
+++ b/barretenberg/cpp/src/barretenberg/proof_system/circuit_builder/ultra_circuit_builder.hpp
@@ -584,6 +584,9 @@ class UltraCircuitBuilder_ : public CircuitBuilderBase<typename Arithmetization_
      * 5) Number of non-native field multiplication gates.
      *
      * @return size_t
+     * TODO(https://github.com/AztecProtocol/barretenberg/issues/875): This method may return an incorrect value before
+     * the circuit is finalized due to a failure to account for "de-duplication" when computing how many
+     * non-native-field gates will be present.
      */
     size_t get_num_gates() const override
     {