On Thu, Aug 10, 2023 at 9:40 AM Richard Biener <richard.guent...@gmail.com> wrote: > > On Thu, Aug 10, 2023 at 3:13 AM liuhongt <hongtao....@intel.com> wrote: > > > > Currently we have 3 different independent tunes for gather > > "use_gather,use_gather_2parts,use_gather_4parts", > > similar for scatter, there're > > "use_scatter,use_scatter_2parts,use_scatter_4parts" > > > > The patch support 2 standardizing options to enable/disable > > vectorization for all gather/scatter instructions. The options is > > interpreted by driver to 3 tunes. > > > > bootstrapped and regtested on x86_64-pc-linux-gnu. > > Ok for trunk? > > I think -mgather/-mscatter are too close to -mfma suggesting they > enable part of an ISA but they won't disable the use of intrinsics > or enable gather/scatter on CPUs where the ISA doesn't have them. > > May I suggest to invent a more generic "short-cut" to > -mtune-ctrl=^X, maybe -mdisable=X? And for gather/scatter > tunables add ^use_gather_any to cover all cases? (or > change what use_gather controls - it seems we changed its > meaning before, and instead add use_gather_8parts and > use_gather_16parts) > > That is, what's the point of this?
https://www.phoronix.com/review/downfall that caused: https://www.phoronix.com/review/intel-downfall-benchmarks Uros.
