[PATCH] D106056: [CVP] processSwitch: Remove default case when switch cover all possible values.
Nathan Chancellor via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 19 11:55:57 PDT 2021
nathanchance added a comment.
@junparser I am not convinced this is a bug in `objtool`. @spatel 's comment makes me wonder if one of the other switch cases is becoming just `return false` because LTO allows LLVM to see that one of the `if (max_contiguous < ...)` checks is always true then this transform removes both that case and the default case because they are equivalent, which seems to be inline with what `objtool` is warning about. I am more than happy to test follow up versions of this patch as I am subscribed, feel free to ping me as well.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106056/new/
https://reviews.llvm.org/D106056
More information about the llvm-commits
mailing list