25/05/2022 13:48, Ray Kinsella: > > Thomas Monjalon <tho...@monjalon.net> writes: > > > The AltiVec header file is defining "vector", except in C++ build. > > The keyword "vector" may conflict easily. > > As a rule, it is better to use the alternative keyword "__vector". > > > > The DPDK header file rte_altivec.h takes care of undefining "vector", > > so the applications and dependencies are free to define the name "vector". > > > > This is a compatibility breakage for applications which were using > > the keyword "vector" for its AltiVec meaning. > > > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > > --- > > doc/guides/rel_notes/release_22_07.rst | 5 +++++ > > lib/eal/ppc/include/rte_altivec.h | 7 +++++++ > > 2 files changed, 12 insertions(+) > > > > Acked-by: Ray Kinsella <m...@ashroe.eu>
Just to make sure, we are all OK to break compatibility of rte_altivec.h? It means the keyword vector is not available anymore with this #include. Please confirm it is OK to merge in DPDK 22.07.