[llvm] [InstSimplify] Add poison propagation for trivially vectorizable intrinsics (PR #149243)
Luke Lau via llvm-commits
llvm-commits at lists.llvm.org
Sat Jul 19 08:34:24 PDT 2025
================
@@ -5213,121 +5213,15 @@ define float @v_exp_f32_dynamic_mode(float %in) #1 {
}
define float @v_exp_f32_undef() {
-; VI-SDAG-LABEL: v_exp_f32_undef:
----------------
lukel97 wrote:
Generally you only need 1 LGTM for uncontroversial changes, but you should use your best judgement if you think a patch is significant enough that it needs input from multiple reviewers. There's some more info in the developer policy docs: https://llvm.org/docs/CodeReview.html#lgtm-how-a-patch-is-accepted
But with this PR as an example, with 1 LGTM from Nikita who owns this part of the code, it should be good to merge!
https://github.com/llvm/llvm-project/pull/149243
More information about the llvm-commits
mailing list