njn | 6b9b9dd | 2004-12-01 11:11:56 +0000 | [diff] [blame] | 1 | ----------------------------------------------------------------------------- |
| 2 | 64-bit cleanness |
| 3 | ----------------------------------------------------------------------------- |
| 4 | The following are places I know or suspect contain code that is not 64-bit |
| 5 | clean. Please mark them off this list as they are fixed, and add any new ones |
| 6 | you know of. |
| 7 | |
| 8 | Memcheck is full of 32-bit assumptions: |
| 9 | - shadow memory in general -- two-level lookup table, etc -- is totally 32-bit |
| 10 | specific. |
| 11 | - new_mem_stack_4 etc are all very 32-bit specific... should make it |
| 12 | new_mem_stack_1wd, etc. Also then need to change |
| 13 | make_aligned_word_noaccess() et al (heaps of places) |
| 14 | - also rotateRight16 and all its dependents... |
njn | be91aae | 2005-03-27 01:42:41 +0000 | [diff] [blame] | 15 | - all the calls to VG_IS_ALIGNED4_ADDR are 32-bit specific code. Likewise |
njn | 6b9b9dd | 2004-12-01 11:11:56 +0000 | [diff] [blame] | 16 | with Addrcheck? Not sure. Plus there's one in vg_transtab.c which doesn't |
| 17 | look right. |
| 18 | |
| 19 | Addrcheck has similar problems. Helgrind too, especially the 32-bit specific |
| 20 | "OTHER_BITS". |
| 21 | |
| 22 | Symbol table stuff: |
njn | 6b9b9dd | 2004-12-01 11:11:56 +0000 | [diff] [blame] | 23 | - symtab -- 32-bit specific stuff in here? eg. search for |
| 24 | "(UInt)sym->st_value" |
| 25 | - stabtype_parser -- all those literal 4's as args to VG_(st_mkint) -- are they |
| 26 | 64-bit clean? Who knows... |
| 27 | |
| 28 | Others: |
| 29 | - search for the following casts: (UInt*), (Int*), (UInt *), (Int *) |
njn | 6b9b9dd | 2004-12-01 11:11:56 +0000 | [diff] [blame] | 30 | - vki_sigset_t.sig use: the debug printing in do_setmask() is wrong -- it |
| 31 | touches sig[1], thus assuming 32-bits. Likewise in handle_SCSS_change() |
| 32 | - client_auxv in vg_main.c is UInt** -- should it be UWord** ? |
| 33 | |