[Lldb-commits] [PATCH] D15357: Update code to silent some ARM/ARM64 specific compiler warnings

Tamas Berghammer via lldb-commits lldb-commits at lists.llvm.org
Wed Dec 9 02:48:26 PST 2015


tberghammer requested changes to this revision.
tberghammer added a comment.
This revision now requires changes to proceed.

We discussed the same change recently (http://reviews.llvm.org/D13866) and the general opinion was that the original code have the right semantics and we don't want to change it just to silence some warnings.

If you want to get rid of these warnings then I would suggest to do it with adding a cast before the comparison (I haven't tested it):

  if ((int)reg_num >= x0 && (int)reg_num <= pc) // We are casting to int to silence a gcc warning


http://reviews.llvm.org/D15357





More information about the lldb-commits mailing list