Tim Armstrong has posted comments on this change.

Change subject: Allow compiler to use SSE3 and SSSE3 instructions.
......................................................................


Patch Set 1:

Compiling with SSE enabled, I don't feel confident that we'll actually get to 
the CpuInfo::VerifyCpuRequirements() check before executing an SSE3 
instruction. Is there some way we can test this or compile the appropriate 
functions/module without enabling SSE3.

-- 
To view, visit http://gerrit.cloudera.org:8080/3291
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib045e00b5aa71cc6ef16dbda160bfdc7b765158a
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Jim Apple <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to