<html>
    <head>
      <base href="https://llvm.org/bugs/" />
    </head>
    <body><table border="1" cellspacing="0" cellpadding="8">
        <tr>
          <th>Bug ID</th>
          <td><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW --- - Tests should not use rand()"
   href="https://llvm.org/bugs/show_bug.cgi?id=31730">31730</a>
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>Tests should not use rand()
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>Test Suite
          </td>
        </tr>

        <tr>
          <th>Version</th>
          <td>trunk
          </td>
        </tr>

        <tr>
          <th>Hardware</th>
          <td>PC
          </td>
        </tr>

        <tr>
          <th>OS</th>
          <td>All
          </td>
        </tr>

        <tr>
          <th>Status</th>
          <td>NEW
          </td>
        </tr>

        <tr>
          <th>Severity</th>
          <td>normal
          </td>
        </tr>

        <tr>
          <th>Priority</th>
          <td>P
          </td>
        </tr>

        <tr>
          <th>Component</th>
          <td>Programs Tests
          </td>
        </tr>

        <tr>
          <th>Assignee</th>
          <td>unassignedbugs@nondot.org
          </td>
        </tr>

        <tr>
          <th>Reporter</th>
          <td>matze@braunis.de
          </td>
        </tr>

        <tr>
          <th>CC</th>
          <td>llvm-bugs@lists.llvm.org
          </td>
        </tr>

        <tr>
          <th>Classification</th>
          <td>Unclassified
          </td>
        </tr></table>
      <p>
        <div>
        <pre>rand() can produce different numbers with different libc implementations.
Therefore we should not use it in the test-suite to get portable tests. Today a
test like this:

$ grep "\<rand\s*()" SingleSource/ MultiSource/ -r

shows 54 affected files. We should probably write a simple everything in one
header  random number generator and replace the rand() usage with that in the
tests.


There is an additional problem of benchmarks that do not even use srand() to
get a deterministic random number sequence from run to run. We spotted this
benchmarking anti-pattern in SingleSource//UnitTests/SignlessTypes/rem.c but
the other benchmarks should be audited for that as well.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are on the CC list for the bug.</li>
      </ul>
    </body>
</html>