[PATCH] D52398: Thread safety analysis: Unwrap __builtin_expect in getTrylockCallExpr

[Haojian Wu via Phabricator via cfe-commits]

hokein added a comment.

Hi, @aaronpuchert, the patch has caused many new warnings in our internal 
codebase, below is a reduced one. Do you mind reverting the patch?

  // if the condition is not true, CHECK will terminate the program.
  #define CHECK(condition) (__builtin_expect((condition), true)) ? 1 : 0
  
  void foo() {
      CHECK(mu.TryLock()); 
      mu.Unlock();
  }


Repository:
  rL LLVM

https://reviews.llvm.org/D52398



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits