[PATCH] D28669: [ARM] Remove ubig32_t cast from ARMAttributeParser

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 13 06:05:24 PST 2017


RKSimon added a comment.

In https://reviews.llvm.org/D28669#645173, @bkramer wrote:

> can we just use support::endian::read32le and support::endian::read32be instead of the casting magic? They do the same thing but should have no problem with implicit conversions in a ternary expression.


Confirmed - this works fine too.


https://reviews.llvm.org/D28669





More information about the llvm-commits mailing list