LGTM, please commit. On Sat, Mar 14, 2015 at 11:11 AM, Szabolcs Sipos < [email protected]> wrote:
> Hi alexfh, > > This patch consists of the suggestions of clang-tidy/misc-static-assert > check. > > http://reviews.llvm.org/D8344 > > Files: > lib/Sema/SemaChecking.cpp > > Index: lib/Sema/SemaChecking.cpp > =================================================================== > --- lib/Sema/SemaChecking.cpp > +++ lib/Sema/SemaChecking.cpp > @@ -1305,9 +1305,10 @@ > // M is C if C is an integer, and ptrdiff_t if C is a pointer, and > // the int parameters are for orderings. > > - assert(AtomicExpr::AO__c11_atomic_init == 0 && > - AtomicExpr::AO__c11_atomic_fetch_xor + 1 == > AtomicExpr::AO__atomic_load > - && "need to update code for modified C11 atomics"); > + static_assert(AtomicExpr::AO__c11_atomic_init == 0 && > + AtomicExpr::AO__c11_atomic_fetch_xor + 1 == > + AtomicExpr::AO__atomic_load, > + "need to update code for modified C11 atomics"); > bool IsC11 = Op >= AtomicExpr::AO__c11_atomic_init && > Op <= AtomicExpr::AO__c11_atomic_fetch_xor; > bool IsN = Op == AtomicExpr::AO__atomic_load_n || > > EMAIL PREFERENCES > http://reviews.llvm.org/settings/panel/emailpreferences/ > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
