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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to