Re: [Qemu-devel] [PATCH 1.1] tci: Fix wrong macro name for debug code

2012-05-08 Thread Anthony Liguori
On 05/03/2012 11:09 AM, Stefan Weil wrote: Code which is compiled with CONFIG_TCG_DEBUG (set by configure option --enable-debug-tcg) should not disable the assert macro by defining NDEBUG. With the wrong macro name CONFIG_TCG_DEBUG, all assertions in tci.c were completely useless because NDEBUG

Re: [Qemu-devel] [PATCH 1.1] tci: Fix wrong macro name for debug code

2012-05-03 Thread Peter Maydell
On 3 May 2012 17:09, Stefan Weil wrote: > Code which is compiled with CONFIG_TCG_DEBUG (set by configure option > --enable-debug-tcg) should not disable the assert macro by > defining NDEBUG. > > With the wrong macro name CONFIG_TCG_DEBUG, all assertions in tci.c > were completely useless because

[Qemu-devel] [PATCH 1.1] tci: Fix wrong macro name for debug code

2012-05-03 Thread Stefan Weil
Code which is compiled with CONFIG_TCG_DEBUG (set by configure option --enable-debug-tcg) should not disable the assert macro by defining NDEBUG. With the wrong macro name CONFIG_TCG_DEBUG, all assertions in tci.c were completely useless because NDEBUG was always defined. Signed-off-by: Stefan We