[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