[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