[mlpack-git] [mlpack] std::size_t warning on 32-bit machines (#553)

[Naman Taneja]

@rcurtin I saw this issue today and tried to summarize, the problem in build warnings. As can be seen from various online sources the Armadillo library when compiled with the C++ 11 sets the default word size to 64bit. The exact code mentioned in the Armadillo package can be found here: http://fossies.org/linux/armadillo/include/armadillo_bits/compiler_setup.hpp.
Another, thing is that the armadillo library is only being used with arma_extend.hpp, which is used to build the cxx objects. Since, we cannot change the distribution code of armadillo, I suggest that we should implement a check to find whether the machine is 32 Bit or 64 Bit in the arma_extend.hpp. Depending upon that we can simply define ARMA_32BIT_WORD and not define ARMA_USE_U64S64. I have checked by removing the ARMA_USE_U64S64 part and including ARMA_32BIT_WORD in the file, which gives a clean build.
If the solution is fine, I can go ahead with the implementation.

---
Reply to this email directly or view it on GitHub:
https://github.com/mlpack/mlpack/issues/553#issuecomment-195122276
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mailman.cc.gatech.edu/pipermail/mlpack-git/attachments/20160310/aed1c57f/attachment.html>