On Mon, Dec 16, 2019 at 7:55 AM Yibo Cai <[email protected]> wrote: > On 12/13/19 7:45 PM, Ravindra Pindikura wrote: > > On Fri, Dec 13, 2019 at 3:41 PM Yibo Cai <[email protected]> wrote: > > > >> Hi, > >> > >> Thanks to pravindra's patch [1], Gandiva loop vectorization is okay now. > >> > >> Will Gandiva detects CPU feature at runtime? My test CPU supports sse to > >> avx2, but I only > >> see "target-features"="+fxsr,+mmx,+sse,+sse2,+x87" in IR, and final code > >> doesn't leverage > >> registers longer than 128. > >> > > > > Can you please give some details about the hardware/OS-version you are > > running this on ? Also, are you building the binaries and running them on > > the same host ? > > > > I'm building and running on same host. > > Build: cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo -DARROW_BUILD_TESTS=ON > -DARROW_GANDIVA=ON .. > > OS: ubuntu 18.04 > > CPU: lscpu outputs below > > Architecture: x86_64 > CPU op-mode(s): 32-bit, 64-bit > Byte Order: Little Endian > CPU(s): 8 > On-line CPU(s) list: 0-7 > Thread(s) per core: 2 > Core(s) per socket: 4 > Socket(s): 1 > NUMA node(s): 1 > Vendor ID: GenuineIntel > CPU family: 6 > Model: 60 > Model name: Intel(R) Core(TM) i7-4790 CPU @ 3.60GHz > Stepping: 3 > CPU MHz: 3591.845 > CPU max MHz: 4000.0000 > CPU min MHz: 800.0000 > BogoMIPS: 7183.72 > Virtualization: VT-x > L1d cache: 32K > L1i cache: 32K > L2 cache: 256K > L3 cache: 8192K > NUMA node0 CPU(s): 0-7 > Flags: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge > mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall > nx pdpe1gb rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl > xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl > vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid sse4_1 sse4_2 x2apic > movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm > cpuid_fault epb invpcid_single pti ssbd ibrs ibpb stibp tpr_shadow vnmi > flexpriority ept vpid fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid > xsaveopt dtherm ida arat pln pts md_clear flush_l1d > > Thanks ! I've a PR to fix this https://github.com/apache/arrow/pull/6038
> > > >> [1] https://github.com/apache/arrow/pull/6019 > >> > > > > > -- Thanks and regards, Ravindra.
