NorenaLeonetti marked 16 inline comments as done. NorenaLeonetti added a comment.
In https://reviews.llvm.org/D33825#772688, @aaron.ballman wrote: > This check is an interesting one. The rules around what is signal safe are > changing for C++17 to be a bit more lenient than what the rules are for > C++14. CERT's rule is written against C++14, and so the current behavior > matches the rule wording. However, the *intent* of the rule is to ensure that > only signal-safe functionality is used from a signal handler, and so from > that perspective, I can imagine a user compiling for C++17 to want the > relaxed rules to still comply with CERT's wording. What do you think? I think your concerns are right. I've checked the upcoming changes in the C++17 standard draft and I'll add some logic to the code to have those relaxed rules at least partly fulfilled. For now, I uploaded the changes required for this check. Repository: rL LLVM https://reviews.llvm.org/D33825 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
