codebase:remove_m32
Differences
This shows you the differences between two versions of the page.
| codebase:remove_m32 [2018/08/23 07:29] – created vodur | codebase:remove_m32 [2018/08/23 07:30] (current) – vodur | ||
|---|---|---|---|
| Line 1: | Line 1: | ||
| - | Sender | + | <code text> |
| - | Date Fri Jun 16 00:49:37 2017~\\ | + | Sender |
| - | Stamp 1497588608\\ | + | Date Fri Jun 16 00:49:37 2017~ |
| - | Expire | + | Stamp |
| - | To imm~\\ | + | Expire |
| - | Subject removing -m32 flag~\\ | + | To imm~ |
| - | Text\\ | + | Subject removing -m32 flag~ |
| - | I propose to remove -m32 flag from makefile.\\ | + | Text |
| - | What we gain from this is mostly future pain of setting up 32-bit build\\ | + | I propose to remove -m32 flag from makefile. |
| - | dependencies on 64-bit systems (as we've all done by now).\\ | + | What we gain from this is mostly future pain of setting up 32-bit build |
| - | Potential issues, as far as I understand, would be if we have code that assumes\\ | + | dependencies on 64-bit systems (as we've all done by now). |
| + | Potential issues, as far as I understand, would be if we have code that assumes | ||
| size of types. Here are the sizes with and without -m32. | size of types. Here are the sizes with and without -m32. | ||
| - | -m32 no -m32\\ | + | |
| - | The size of int 4 4\\ | + | The size of int |
| - | The size of short 2 2\\ | + | The size of short |
| - | The size of long 4 8\\ | + | The size of long 4 8 |
| - | The size of float 4 4\\ | + | The size of float |
| - | The size of double | + | The size of double |
| - | The size of long double 12 16\\ | + | The size of long double 12 16 |
| - | The size of char 1 1\\ | + | The size of char 1 1 |
| - | The size of enum 4 4\\ | + | The size of enum 4 4 |
| - | The size of pointer | + | The size of pointer |
| - | So only long, long double, and pointer sizes have changed. This being the case\\ | + | So only long, long double, and pointer sizes have changed. This being the case |
| I say the risk of issues from size change is low. | I say the risk of issues from size change is low. | ||
| - | I verified code can compile and run just fine without -m32. Only issue I see at\\ | + | I verified code can compile and run just fine without -m32. Only issue I see at |
| - | the moment is there is casting between void * and int in hunt.c which gets a\\ | + | the moment is there is casting between void * and int in hunt.c which gets a |
| warning since they are different sizes with no -m32. | warning since they are different sizes with no -m32. | ||
| Thoughts? | Thoughts? | ||
| - | P.S.: On a related note, we are currently using gcc 6.2.1 on the host right now\\ | + | P.S.: On a related note, we are currently using gcc 6.2.1 on the host right now |
| - | (through scl devtoolset-6), | + | (through scl devtoolset-6), |
| - | so if/when we do start adding C++ code, we would need to turn -m32 off or go\\ | + | so if/when we do start adding C++ code, we would need to turn -m32 off or go |
| - | back to gcc 4.1.2.\\ | + | back to gcc 4.1.2. |
| ~ | ~ | ||
| + | </ | ||
codebase/remove_m32.1535009385.txt.gz · Last modified: 2018/08/23 07:29 by vodur