[PATCH] D87962: [clang] Change the multi-character character constants from extension to implementation-defined.
Richard Smith - zygoloid via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 1 22:28:13 PDT 2020
rsmith added a comment.
In D87962#2306043 <https://reviews.llvm.org/D87962#2306043>, @aaron.ballman wrote:
> That doesn't sound like the right approach to me -- Remarks are usually for reporting backend decision-making to the frontend for things like optimization passes.
To be clear: that's how they happen to most visibly be used, but the more general idea is that Remarks are for purely informational messages. One test for whether a diagnostic could reasonably be a remark is: can we imagine anyone ever reasonably wanting to promote it to an error as part of the flags for some project? If so, then use of a remark is inappropriate. That's certainly the case here: it's entirely reasonable to want to be able to reject use of non-portable functionality such as this. So I agree, this should not be a remark.
> Btw, it looks like when you generated this patch, it was generated against the previous diff and not trunk, so the diff view is misleading. When you update the patch, can you be sure to diff against the trunk?
Please also ensure you upload a diff with full context <http://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface>.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87962/new/
https://reviews.llvm.org/D87962
More information about the cfe-commits
mailing list