Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

[Windows][clang] 6369267 does not compile: -Werror,-Wnull-conversion in qwen3_5.cpp (pointer passed as bool return_hidden)

Abierto Apto para principiantes
  • ¿Algo no está bien?
#3,308 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
82/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
cpp
Área
machine-learning

Línea de trabajo

Comience en src/vllm/model_executor/models/qwen3_5.cpp, en la firma de DenseForwardLayers alrededor de las líneas 9822-9836 y en la llamada alrededor de la línea 12483. Compare las posiciones de los argumentos y determine si la llamada debe pasar un booleano o un puntero al dispositivo; después, verifique la llamada corregida con la compilación de Windows clang usando -Wall -Wextra -Werror.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Summary

Commit 636926736c0b6053eda99d08c1f753b29938fac0 does not compile with clang on Windows
(the tree builds with -Wall -Wextra -Werror):

src/vllm/model_executor/models/qwen3_5.cpp:12483:65: error: implicit conversion of nullptr constant to 'bool' [-Werror,-Wnull-conversion]
 12480 |       lg = DenseForwardLayers(d, s.hidden->t(), s.positions, s.attn_meta,
       |            ~~~~~~~~~~~~~~~~~~
 12481 |                               s.gdn_meta, attn_kv, gdn_state, impl_->weights,
 12482 |                               impl_->config, {}, nullptr, nullptr, aux_ids_arg,
 12483 |                               aux_out_arg, dbuf ? s.dev.get() : nullptr);
       |                                                                 ^~~~~~~
       |                                                                 false
1 error generated.
Why it fires

The call passes 15 arguments; the 15th parameter of DenseForwardLayers is
bool return_hidden (signature at src/vllm/model_executor/models/qwen3_5.cpp:9822-9836),
so the pointer expression dbuf ? s.dev.get() : nullptr is converted to bool. clang
rejects the null-constant arm under -Werror,-Wnull-conversion.

Passing s.dev.get() as return_hidden also looks unintended at the call site — the 16th
parameter, StepDevInputs* persistent_sdi, is defaulted here, which suggests a reshuffled
argument list rather than an intentional value.

Environment
  • Windows 11 (26200), native Windows build with clang/lld from TheRock 10.0, HIP gfx1151
  • commit 6369267, -O3 -DNDEBUG -g -Xclang -gcodeview, --target server
  • 9e63db5d compiles fine, so this was introduced by one of the 36 commits in between
Local workaround (to get a build)

static_cast<bool>(dbuf ? s.dev.get() : nullptr) — behaviour-preserving, but it does not
fix the likely wrong argument.

Suggested change

Pass the intended boolean (e.g. dbuf != nullptr) or the intended device argument, and make
the call site match the declared parameter list.

Lenguaje dominante
C++
Estrellas
423
Forks
53
Merge medio
1 d 7 h
PR fusionados (30 d)
382

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de mudler/vllm.cpp

Todos los issues de mudler/vllm.cpp

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.