> -----Original Message----- > From: David Marchand <[email protected]> > Sent: Friday, March 20, 2026 1:33 PM > To: [email protected] > Cc: [email protected]; Dumitrescu, Cristian <[email protected]> > Subject: [PATCH] pipeline: fix build with sanitizers or debug options > > Similar to commit 84f5ac9418ea ("pipeline: fix build with ASan"). > > Here we are again. Depending on options (like debug, or ASan, or UBSan), > compilation can fail because of dumb construct like CHECK(0, XXX). > Dumb, because such an expression macro expands as: if (0) return -XXX; > > ../lib/pipeline/rte_swx_pipeline.c: In function ‘instr_movh_translate’: > ../lib/pipeline/rte_swx_pipeline.c:3461:1: error: control reaches end of > non-void function [-Werror=return-type] > 3461 | } > | ^ > > Remove any such call when at the end of functions, using a regexp: > %s/CHECK(0, \(.*\))\(;\n}\)/return -\1\2/ > > Cc: [email protected] > > Signed-off-by: David Marchand <[email protected]> > ---
Acked-by: Cristian Dumitrescu <[email protected]>
