aaron.ballman added inline comments.

================
Comment at: lib/Sema/SemaTemplate.cpp:3055
 
+// Print a diagnostic for the failing static_assert expression. Defaults to
+// pretty-printing the expression.
----------------
Comment is a bit out of date as this is no longer specific to `static_assert`.

It looks like this may also change the behavior of enable_if diagnostic 
reporting. Do you need to update any of those tests from this change? If not, 
can you devise some tests for that case as well to show that this isn't just a 
static_assert behavior? (Do a search for `findFailedBooleanCondition` to find 
where the behavior has changed.)


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54903/new/

https://reviews.llvm.org/D54903



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

Reply via email to