[PATCH] D48322: [Sema] Discarded statment should be an evaluatable context

2018-07-03 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL336233: [Sema] Discarded statment should be an evaluatable context. (authored by epilk, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D48322?

[PATCH] D48322: [Sema] Discarded statment should be an evaluatable context

2018-07-03 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. In https://reviews.llvm.org/D48322#1150316, @erik.pilkington wrote: > Is this what you were concerned about? Yes, exactly. Thank you for checking. Repository: rC Clang https://reviews.ll

[PATCH] D48322: [Sema] Discarded statment should be an evaluatable context

2018-07-02 Thread Erik Pilkington via Phabricator via cfe-commits
erik.pilkington added a comment. In https://reviews.llvm.org/D48322#1148424, @rsmith wrote: > Hmm, so this will mean that we can have internal linkage declarations marked > `Used` for which there is no definition, and we need to not warn on that. > > I think it might be better to avoid marking t

[PATCH] D48322: [Sema] Discarded statment should be an evaluatable context

2018-06-29 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. Hmm, so this will mean that we can have internal linkage declarations marked `Used` for which there is no definition, and we need to not warn on that. I think it might be better to avoid marking things used in this case (even though they're still technically odr-used).

[PATCH] D48322: [Sema] Discarded statment should be an evaluatable context

2018-06-29 Thread Erik Pilkington via Phabricator via cfe-commits
erik.pilkington added a comment. Herald added a subscriber: dexonsmith. Ping! Repository: rC Clang https://reviews.llvm.org/D48322 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D48322: [Sema] Discarded statment should be an evaluatable context

2018-06-19 Thread Erik Pilkington via Phabricator via cfe-commits
erik.pilkington created this revision. erik.pilkington added a reviewer: rsmith. The constexpr evaluator was erroring out because these templates weren't defined. Despite being used in a discarded statement, we still need to constexpr evaluate them, which means that we need to instantiate them.