<div dir="ltr"><div dir="ltr">On Tue, Feb 11, 2020 at 8:22 AM Teresa Johnson <<a href="mailto:tejohnson@google.com">tejohnson@google.com</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Feb 10, 2020 at 11:59 PM Mikael Holmén via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Peter,<br>
<br>
I get two warnings with this commit when compiling with clang 8.0:<br>
<br>
/repo/uabelho/master/compiler-<br>
rt/lib/scudo/standalone/size_class_map.h:122:20: error: implicit<br>
conversion loses integer precision: 'unsigned long' to 'scudo::u8' (aka<br>
'unsigned char') [-Werror,-Wimplicit-int-conversion]<br>
          return i + 1;<br>
          ~~~~~~ ~~^~~<br>
/repo/uabelho/master/compiler-<br>
rt/lib/scudo/standalone/size_class_map.h:124:14: error: implicit<br>
conversion changes signedness: 'int' to 'scudo::u8' (aka 'unsigned<br>
char') [-Werror,-Wsign-conversion]<br>
      return -1;<br>
      ~~~~~~ ^~<br>
2 errors generated.<br></blockquote><div><br></div><div>I get these as well, and when I work around them by adding casts I get an additional failure:</div><div><br></div>compiler-rt/lib/scudo/standalone/size_class_map.h:27:21: error: use of undeclared identifier 'MaxSize'<br>    DCHECK_LE(Size, MaxSize);<br></div></div></blockquote><div><br></div><div>These should all be fixed by 515e19ae7b5.</div><div><br></div><div>Peter</div></div></div>