[PATCH] D50623: [NFC] [Hexagon] Simplify int8 truncation and validation

Kim Gräsman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 13 01:30:59 PDT 2018


kimgr added a comment.

The original code should have worked (`Opc` was constrained to `storerb`, `storerh` or `storeri` by validation earlier in the method), but GCC complained:

  llvm/lib/Target/Hexagon/HexagonBitSimplify.cpp:1989:16: warning: ‘V’ is used uninitialized in this function [-Wuninitialized]

Hopefully the new code handles truncation correctly -- `ninja check-llvm` passes for me.


Repository:
  rL LLVM

https://reviews.llvm.org/D50623





More information about the llvm-commits mailing list