[PATCH] D22441: Fix isShiftedInt and isShiftedUint for widths > 32.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 18 15:45:14 PDT 2016


RKSimon added a comment.

In https://reviews.llvm.org/D22441#487373, @jlebar wrote:

> RKSimon, can you try https://reviews.llvm.org/D22472?  I'm going to close this revision again, otherwise it's confusing to have an open revision after the patch has landed.  Let's take further discussion into that thread, if that's OK.


Thanks its fixed the shift count warning but now has an operator precedence warning.

If you're after a buildbot to check things, llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast will help. It doesn't treat warnings as errors so the build stays green and you need to drill down into the build stdio:

http://lab.llvm.org:8011/builders/llvm-clang-lld-x86_64-scei-ps4-windows10pro-fast/builds/8507/steps/build/logs/warnings%20%283%29


Repository:
  rL LLVM

https://reviews.llvm.org/D22441





More information about the llvm-commits mailing list