[PATCH] D86301: [Verifier] Additional check for get.active.lane.mask
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 25 15:14:13 PDT 2020
efriedma added a comment.
This check isn't appropriate. Given that the second argument to @llvm.get.active.lane.mask.v4i1.i32 is a variable, in general, it isn't appropriate to print an error in cases where we can prove the value is 0. You can add a Lint check if you want.
Also, if `@llvm.get.active.lane.mask.v4i1.i32(i32 0, i32 0)` is supposed to produce poison, that isn't documented at all.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D86301/new/
https://reviews.llvm.org/D86301
More information about the llvm-commits
mailing list