Re: [PATCH 01/10] eal: add workaround for __builtin_constant_p

2025-02-11 Thread fengchengwen
Signed-off-by: Chengwen Feng On 2025/2/12 6:01, Andre Muezerie wrote: > There's no MSVC equivalent for compiler extension __builtin_constant_p. > EAL already had __rte_constant which was used as a first attempt to > workaround __builtin_constant_p when using MSVC. However, there are > pieces of c

[PATCH 01/10] eal: add workaround for __builtin_constant_p

2025-02-11 Thread Andre Muezerie
There's no MSVC equivalent for compiler extension __builtin_constant_p. EAL already had __rte_constant which was used as a first attempt to workaround __builtin_constant_p when using MSVC. However, there are pieces of code that would benefit from being able to provide a default value to be used ins

Re: [PATCH 01/10] eal: add workaround for __builtin_constant_p

2025-02-11 Thread Stephen Hemminger
On Tue, 11 Feb 2025 14:01:57 -0800 Andre Muezerie wrote: > There's no MSVC equivalent for compiler extension __builtin_constant_p. > EAL already had __rte_constant which was used as a first attempt to > workaround __builtin_constant_p when using MSVC. However, there are > pieces of code that woul