diff options
| author | Ben Skeggs <skeggsb@gmail.com> | 2008-06-12 23:49:26 +1000 |
|---|---|---|
| committer | Ben Skeggs <skeggsb@gmail.com> | 2008-06-29 15:46:18 +1000 |
| commit | 2fdeb4d5a5cc8b93bf885ba646e3a29a68c755ed (patch) | |
| tree | 375e8267a5a58ebbc5243f51f2cf539fdc3766ec | |
| parent | 4d520e0b76cf54ae8eb5464afc126c6cc5c6bfdc (diff) | |
nv50: comment on a so-far unseen bug
| -rw-r--r-- | src/gallium/drivers/nv50/nv50_program.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/gallium/drivers/nv50/nv50_program.c b/src/gallium/drivers/nv50/nv50_program.c index af5ca84c8e..39597c5481 100644 --- a/src/gallium/drivers/nv50/nv50_program.c +++ b/src/gallium/drivers/nv50/nv50_program.c @@ -22,6 +22,9 @@ * - Maybe even relax restrictions a bit, can't do P_RESULT + P_IMMD, * but can emit to P_TEMP first - then MOV later. NVIDIA does this * + * In ops such as ADD it's possible to construct a bad opcode in the !is_long() + * case, if the emit_src() causes the inst to suddenly become long. + * * Verify half-insns work where expected - and force disable them where they * don't work - MUL has it forcibly disabled atm as it fixes POW.. * |