[PATCH] D85759: [SLPVectorizer] Fix regression from cost model refactoring
Thomas Lively via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 11 12:20:40 PDT 2020
tlively added a comment.
In D85759#2211133 <https://reviews.llvm.org/D85759#2211133>, @fhahn wrote:
> In D85759#2211128 <https://reviews.llvm.org/D85759#2211128>, @tlively wrote:
>
>> Huh, the precommitted test failed on multiple bots even though it passes for me locally. I've reverted the commit that added the test for now while I investigate.
>
> I think you have to add a REQUIRES for the WebAssembly target, otherwise bots that do not build that target will fail with the wasm32 triple.
Yes, thanks for the catch!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85759/new/
https://reviews.llvm.org/D85759
More information about the llvm-commits
mailing list