Let&#39;s follow the precedent.  Please correct. <br><br>If you are feeling generous, the Test in 6837094 could be fixed, too.<br><br>Thanks.<br><br><div class="gmail_quote">On Thu, Jul 16, 2009 at 11:42 AM, Tom Rodriguez <span dir="ltr">&lt;<a href="mailto:Thomas.Rodriguez@sun.com">Thomas.Rodriguez@sun.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Thanks for the explanation.  I&#39;m happy to correct it if you like.<br><font color="#888888">
<br>
tom</font><div><div></div><div class="h5"><br>
<br>
On Jul 16, 2009, at 7:22 AM, Martin Buchholz wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
We&#39;re still working out legalities, but the precedent for legal notices<br>
for a source file that is non-Sun-authored<br>
is to not include the Sun copyright line<br>
<br>
Copyright 2009 Sun Microsystems, Inc.<br>
<br>
For example, here is a Google-authored file:<br>
<br>
<a href="http://cr.openjdk.java.net/%7Emartin/webrevs/openjdk7/timsort/src/share/classes/java/util/TimSort.java.html" target="_blank">http://cr.openjdk.java.net/~martin/webrevs/openjdk7/timsort/src/share/classes/java/util/TimSort.java.html</a><br>

<br>
(and files obtained from the public domain have no copyright line<br>
inside their legal notice at all)<br>
<br>
It&#39;s no big deal either way, especially for tests,<br>
as all contributors retain their rights under SCA.<br>
<br>
Martin<br>
<br>
On Wed, Jul 15, 2009 at 13:37, Tom Rodriguez &lt;<a href="mailto:Thomas.Rodriguez@sun.com" target="_blank">Thomas.Rodriguez@sun.com</a>&gt; wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<a href="http://cr.openjdk.java.net/%7Enever/6860469" target="_blank">http://cr.openjdk.java.net/~never/6860469</a> is what I&#39;m going to push with you and hiroshi marked as contributors.  I added the copyright and fixed the test case run running with jtreg.  The class needed to be public and the class name was missing from the @run line.<br>

<br>
tom<br>
<br>
On Jul 15, 2009, at 12:24 PM, Chuck Rasbold wrote:<br>
<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I presume that Sun will add the copyright notice when the code is committed, and of course, I&#39;m OK with that.<br>
<br>
IANAL, but I&#39;m reluctant to add the notice in advance, mostly because I am not employed by Sun.<br>
<br>
On Wed, Jul 15, 2009 at 12:07 PM, Tom Rodriguez &lt;<a href="mailto:Thomas.Rodriguez@sun.com" target="_blank">Thomas.Rodriguez@sun.com</a>&gt; wrote:<br>
The test is missing the copyright notice but otherwise this looks good.  BTW, I ran a full ctw with this change both as is and as an assert that checked for cases where this would produce a different control and I only found 4 cases where it ever changed the answer and none of those triggered a failure in the way the test case does.  Hiroshi, thanks for tracking this down and Chuck, thanks for distilling a test case.<br>

<br>
tom<br>
<br>
<br>
On Jul 15, 2009, at 9:27 AM, Chuck Rasbold wrote:<br>
<br>
<a href="http://cr.openjdk.java.net/%7Erasbold/6860469/webrev.00" target="_blank">http://cr.openjdk.java.net/~rasbold/6860469/webrev.00</a><br>
<br>
Fixed 6860469: remix_address_expressions sets incorrect control causing crash in split_if_with_block_post<br>
<br>
Consult the control node of both inputs when deciding where to place<br>
new LShiftI node.<br>
<br>
Previously, we assumed the invar input should set the new node&#39;s<br>
control, and the scale input was irrelevant for control&#39;s sake.<br>
However, sometimes the invar input is a constant, allowing it to be<br>
higher in the dom tree than the scale.<br>
<br>
Fix provided by Hiroshi Yamauchi (<a href="mailto:yamauchi@google.com" target="_blank">yamauchi@google.com</a>)<br>
<br>
<br>
<br>
</blockquote>
<br>
</blockquote></blockquote>
<br>
</div></div></blockquote></div><br>