Skip to content

Commit f21af88

Browse files
committed
[None][chore] Remove file-level summary comment from encDecBeamSearchTest
The top-of-file block described the bug fix in PR-specific terms ("Before the fix"/"After the fix") that have no long-term value in the source tree. The per-test docstring already captures the invariant being verified. Signed-off-by: Aurelien Chartier <2567591+achartier@users.noreply.github.com>
1 parent abc3c42 commit f21af88

1 file changed

Lines changed: 0 additions & 10 deletions

File tree

cpp/tests/unit_tests/batch_manager/encDecBeamSearchTest.cpp

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,16 +15,6 @@
1515
* limitations under the License.
1616
*/
1717

18-
// Regression tests for encoder-decoder beam-search logits gathering.
19-
//
20-
// fragmentPointerDevice is per-batch-slot ([maxBatchSize, kCACHE_LENGTH]).
21-
// Before the fix, fragmentPointerDevice was a single shared row ([kCACHE_LENGTH]); sequential
22-
// flushes from different requests in the same batch clobbered each other's GPU pointer
23-
// arrays, causing the mergeLogitsFragmentsKernel to read stale fragment addresses and
24-
// produce degenerate output with gather_generation_logits=True.
25-
// After the fix, each request gets its own device-side pointer row via getFragmentPointerSlot(),
26-
// eliminating cross-request interference.
27-
2818
#include "tensorrt_llm/batch_manager/kvCacheManager.h"
2919
#include "tensorrt_llm/batch_manager/llmRequest.h"
3020
#include "tensorrt_llm/batch_manager/runtimeBuffers.h"

0 commit comments

Comments
 (0)