<html>
    <head>
      <base href="https://bugs.llvm.org/">
    </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 - Thread safety analysis: False negatives for exclusive/shared joins on back edges"
   href="https://bugs.llvm.org/show_bug.cgi?id=51913">51913</a>
          </td>
        </tr>

        <tr>
          <th>Summary</th>
          <td>Thread safety analysis: False negatives for exclusive/shared joins on back edges
          </td>
        </tr>

        <tr>
          <th>Product</th>
          <td>clang
          </td>
        </tr>

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

        <tr>
          <th>Hardware</th>
          <td>All
          </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>Frontend
          </td>
        </tr>

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

        <tr>
          <th>Reporter</th>
          <td>aaronpuchert@alice-dsl.net
          </td>
        </tr>

        <tr>
          <th>CC</th>
          <td>llvm-bugs@lists.llvm.org, neeilans@live.com, richard-llvm@metafoo.co.uk
          </td>
        </tr></table>
      <p>
        <div>
        <pre>Since <a href="https://reviews.llvm.org/D102026">https://reviews.llvm.org/D102026</a> we don't warn on this code:

// <a href="https://clang.llvm.org/docs/ThreadSafetyAnalysis.html#mutex-h">https://clang.llvm.org/docs/ThreadSafetyAnalysis.html#mutex-h</a>
#include "mutex.h"

Mutex mu;
int x GUARDED_BY(mu);

void f() {
  MutexLocker scope(&mu);
  for (unsigned i = 1; i < 10; ++i) {
    x = 1;
    if (i == 5) {
      scope.Unlock();
      scope.ReaderLock();
    }
  }
}

On the first iteration we have mu exclusively and can modify x, but on later
iterations we might not. So we can't allow this join to be silent.</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>