Re: [PATCH] D16973: Fix ICE with constexpr and friend functions

2016-02-12 Thread Olivier Goffart via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL260675: Fix ICE with constexpr and friend functions (authored by ogoffart). Changed prior to commit: http://reviews.llvm.org/D16973?vs=47146&id=47791#toc Repository: rL LLVM http://reviews.llvm.org/

Re: [PATCH] D16973: Fix ICE with constexpr and friend functions

2016-02-11 Thread Richard Smith via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM, thanks! http://reviews.llvm.org/D16973 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listin

Re: [PATCH] D16973: Fix ICE with constexpr and friend functions

2016-02-11 Thread Olivier Goffart via cfe-commits
ogoffart added a comment. ping? http://reviews.llvm.org/D16973 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Re: [PATCH] D16973: Fix ICE with constexpr and friend functions

2016-02-07 Thread Olivier Goffart via cfe-commits
ogoffart updated this revision to Diff 47146. ogoffart added a comment. Optimized by avoiding repeated call to getBody http://reviews.llvm.org/D16973 Files: lib/AST/ExprConstant.cpp test/SemaCXX/constant-expression-cxx11.cpp Index: test/SemaCXX/constant-expression-cxx11.cpp ===