<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sun, Mar 13, 2016 at 12:23 AM, Joerg Sonnenberger via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, Mar 13, 2016 at 12:10:51AM -0800, Rui Ueyama wrote:<br>
> Is there any case you want to use --warn-common option when linking Fortran<br>
> programs?<br>
<br>
</span>I still think it is useful it only warns for different sizes, but if it<br>
is warnings even when the common definitions are consistent, it is<br>
useless for that.</blockquote><div><br></div><div>By "that", do you mean Fortran?</div><div><br></div><div>Anyways, I think this patch is okay. This is consistent -- it warns every time a common symbol is merged with not exceptions. It is easy to understand, and in addition to that, it's compatible with GNU linkers.</div><div><br></div><div>The other way is to tell the users to use -fno-commons and don't submit this patch completely, but that's I guess a bit too much.</div></div></div></div>