[llvm] [Target] Change `no-trap-after-noreturn` to `trap-after-noreturn` (PR #67925)
Fangrui Song via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 3 01:11:23 PDT 2023
MaskRay wrote:
> OK. If the naming was intentional, I'm fine keeping it as is.
I think a variable with a positive meaning (`TrapAfterNoreturn`) is preferred. You could make the change non-functional.
It's just that how `TrapAfterNoreturn` is set to true and false should be thought carefully.
https://github.com/llvm/llvm-project/pull/67925
More information about the llvm-commits
mailing list