[PATCH] D36806: Switch to cantFail(), since it does the same assertion.

Stephen Hines via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 30 19:05:22 PDT 2017

Previous message: [PATCH] D36806: Switch to cantFail(), since it does the same assertion.
Next message: [PATCH] D37122: Change Diagnostic Category size error from runtime to compiletime
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

srhines added a comment.

In https://reviews.llvm.org/D36806#856070, @lhames wrote:

> I've added an optional ErrMsg argument to cantFail in r312066 - you can now use this to preserve your explanatory text.


Thank you for adding this! I updated the CL to make use of it.


https://reviews.llvm.org/D36806

Previous message: [PATCH] D36806: Switch to cantFail(), since it does the same assertion.
Next message: [PATCH] D37122: Change Diagnostic Category size error from runtime to compiletime
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the cfe-commits mailing list