2017-05-11 17:32 GMT+02:00 Matteo Bruni matteo.mystral@gmail.com:
2017-05-11 17:08 GMT+02:00 Paul Gofman gofmanp@gmail.com:
The problem is I met this opcode in the wild and thus added it in the first version of preshader implementation (and can "model" and test it by manually editing preshader code), but could not make native d3dx compiler to generate it so far. Whatever I tried in HLSL which would be a straightforward use of 'cmp' resulted in different opcode sequences. It was either generated by some earlier versions of compiler or I just could not find the conditions under which it will be used. As for adding some other 3-parameter opcode to effect blob, while scanning all the opcodes in the 3 hex digit opcode range I have found just one other 3 parameter opcode 'movc', but it seem to always return 0 and I cannot generate it in effect blob from source either. So by now I didn't find any other way to test this opcode rather than manually code it in the effect blob.
Maybe I would rather generate the code for all the other cases too? Itwon't make this part less evident but it will be a uniform approach.
Well, yes, I mean creating by hand an effect blob with the specific instruction encodings you're currently testing in test_effect_preshader_ops(). You might make a separate preshader / state for each instruction or not (or anything in between), whatever is easier.
Not sure we're on the same page so let me try to say it differently: what I'd like to see is a hand-crafted effect blob which contains preshader(s) which use those "interesting" instructions. That would be used in a test replacing test_effect_preshader_ops().
Hand-crafting an effect file is probably going to be a bit tedious (e.g. the typedefs can get quite convoluted) so I suggest you to start by picking a few pieces of one of the existing effect blobs (e.g. test_effect_preshader_effect_blob[]) and fixing up the offsets and counts.