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

[Sam Parker via Phabricator via llvm-commits]

samparker added inline comments.


================
Comment at: lib/Support/ARMAttributeParser.cpp:689
+    uint32_t SectionLength =
+      *reinterpret_cast<const support::ulittle32_t*>(Section.data() + Offset);
 
----------------
RKSimon wrote:
> If you replace the ternary operator with an if - else pair - then MSVC would be happy with the casts and you keep endian support
ah, ok! thanks!


https://reviews.llvm.org/D28669