westonpace commented on code in PR #13218:
URL: https://github.com/apache/arrow/pull/13218#discussion_r888571835
##########
cpp/src/arrow/compute/light_array.h:
##########
@@ -31,6 +33,18 @@
namespace arrow {
namespace compute {
+/// \brief Context needed by various execution engine operations
+///
+/// In the execution engine this context is provided by either the node or the
+/// plan and the context exists for the lifetime of the plan. Defining this
here
+/// allows us to take advantage of these resources without coupling the logic
with
+/// the execution engine.
+struct LightContext {
+ bool has_avx2() const { return (hardware_flags &
arrow::internal::CpuInfo::AVX2) > 0; }
Review Comment:
Leaving this alone for now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
