[GLUTEN-7950][VL] Keep Core module's build flag consistent with Velox

[surnaik]

1. Keep build flags consistent with Velox Module in cpp/velox/CMakeList.txt
2. Keep velox build flag to default to increase coverage of targeted CPU Platforms

How was this patch tested?

Local Build and tested on multiple GCE VMs - N1, N2, C2, N2D, C2D etc

[github-actions]

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

• Other pull requests

[surnaik]

@PHILO-HE / @FelixYBW please review. Thanks!

[PHILO-HE]

@surnaik, thanks so much for your pr and test feedback! We already have a draft pr to fix this issue: #7916.
And I think we should set flags in root CMakeLists.txt to make them applied consistently across all submodules.
cc @zhouyuan

[surnaik]

@surnaik, thanks so much for your pr and test feedback! We already have a draft pr to fix this issue: #7916. And I think we should set flags in root CMakeLists.txt to make them applied consistently across all submodules. cc @zhouyuan

Thanks @PHILO-HE , You may please close this PR if Yuan is already fixing it, didn't notice it.

[github-actions]

#7950

[zhouyuan]

👍
thanks for the fix, looks good to me. We are discussing if we could make another profile so some AVX512 instructions can be used inside Gluten. I will try to improve more on this part

[PHILO-HE]

Let's land this pr firstly. @surnaik, thanks so much for your work!

[surnaik]

Thanks @PHILO-HE , as you suggested, let me actually clean it up and add it to the top level CMakeList.txt

[zhouyuan]

#8041