[llvm-commits] patch: CodeGen/BinaryObject.h explicit typecast

Chris Lattner clattner at apple.com
Tue Jul 7 11:29:16 PDT 2009


On Jul 7, 2009, at 7:46 AM, Ryan Flynn wrote:

> BinaryObject.h was implicitly converting between uint{16,32,64}_t to
> uint8_t (via 'foo & 255'), i replaced this with an explicit (uint8_t)
> cast which is equivalent, faster and more correct (silences
> type-related warnings). Also, following coding standards I replaced
> post-increment with pre-increment.

Looks great, thanks!
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090706/080244.html

-Chris



More information about the llvm-commits mailing list