[PATCH] D26506: Compiler-rt part of D26230: Add (constant) masked load/store support
Kuba (Brecka) Mracek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 2 11:59:30 PST 2016
kubabrecka added a comment.
In https://reviews.llvm.org/D26506#611883, @filcab wrote:
> I've tried pushing this test, but one of the buildbots (at least) doesn't have AVX support. And AVX/AVX2 are the only things on X86 that will generate those masked loads/stores.
> I'd go with either:
>
> No testing on compiler-rt (ASan doesn't behave differently in the face of masked ops, so we should be able to go by with only the tests from llvm)
> Testing on compiler-rt using IR, *assuming* we can lower calls to masked.loads/stores even on non-AVX CPUs
> Testing on compiler-rt using some other arch that's not X86 (I don't really have the hardware to try that out, though)
Can we just add "REQUIRES: avx" to the test and lit?
Repository:
rL LLVM
https://reviews.llvm.org/D26506
More information about the llvm-commits
mailing list