Add tzcnt as an acceptable instruction to x86-64 validator (it already

Add tzcnt as an acceptable instruction to x86-64 validator (it already
handled lzcnt).

BUG= http://code.google.com/p/nativeclient/issues/detail?id=2869
Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=9278

[Karl, 2012-07-24 21:03:02]

Need to change lzcnt and tzcnt to work as 386-instruction.

See comments by Roland on CL http://codereview.chromium.org/10809021 for further
details.

[Karl, 2012-07-24 21:28:18]

http://codereview.chromium.org/10790077/diff/10004/src/trusted/validator/x86/...
File src/trusted/validator/x86/decoder/generator/ncdecode_sse.c (right):

http://codereview.chromium.org/10790077/diff/10004/src/trusted/validator/x86/...
src/trusted/validator/x86/decoder/generator/ncdecode_sse.c:337: NaClDefine("  
f30fbc:     Tzcnt $Gv, $Ev",      NACLi_386,     st, Move);
Made tzcnt and lzcnt work on all processors.

[Karl, 2012-07-26 19:31:25]

Is anyone reviewing this CL?

[Brad Chen, 2012-07-26 19:40:19]

Oops; I'll have a look today.

On Thu, Jul 26, 2012 at 12:31 PM, <kschimpf@google.com> wrote:

> Is anyone reviewing this CL?
>
>
https://chromiumcodereview.**appspot.com/10790077/<https://chromiumcodereview...
>

-- 
You received this message because you are subscribed to the Google Groups
"Native-Client-Reviews" group.
To post to this group, send email to native-client-reviews@googlegroups.com.
To unsubscribe from this group, send email to
native-client-reviews+unsubscribe@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/native-client-reviews?hl=en.

[Brad Chen, 2012-07-26 19:44:38]

LGTM

Why is it that the production 32-bit validator doens't need this fix?

[Karl, 2012-07-26 19:48:21]

The fix for the 32-bit version is a separate CL.