[llvm] [Target] Change `no-trap-after-noreturn` to `trap-after-noreturn` (PR #67925)

Artem Belevich via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 2 12:40:03 PDT 2023


Artem-B wrote:

> Please add a patch description justifying this change. As-is, this seems like a pointless rename to me.

I've suggested renaming it to improve readability. Double negations in a single word are already hard to read, and when it's used in a negation, `!NoTrapAfterNoreturn` looks outright odd to me.

We don't need these changes, but I think it's a nice cleanup.


https://github.com/llvm/llvm-project/pull/67925


More information about the llvm-commits mailing list