<div dir="ltr">But we already have the same assignment in X86_64TargetInfo<ELFT>::relocateOne, so I guess we've accepted that?</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 26, 2017 at 12:51 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>================<br>
>Comment at: lld/trunk/ELF/Target.cpp:518<br>
>+  if (Type == R_386_8 || Type == R_386_PC8) {<br>
>+    *Loc = (uint8_t)Val;<br>
>+    return;<br>
>----------------<br>
>Do you need a cast?<br>
<br>
</span>I think we want it.<br>
<br>
Otherwise with clang's -Weverything/-Wconversion it should give something like<br>
"warning: implicit conversion loses integer precision: 'uint64_t' (aka 'unsigned long') to 'uint8_t' (aka 'unsigned char') [-Wconversion]"<br>
<span class="HOEnZb"><font color="#888888"><br>
George.<br>
<br>
<br>
<br>
</font></span></blockquote></div><br></div>