[PATCH] D68064: [Builtins] Provide a mechanism to selectively disable tests based on whether an implementation is provided by a builtin library.

Dan Liew via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 10:58:05 PDT 2019


delcypher added a comment.

Okay I figured out the issue. The problem is with the filtering code. It assumes that architecture specific specializations have a filepath like `<arch>/filename`. The problem here is that for `powerpc64` the `<arch>` is `powerpc64` but the directory name used is actually `ppc`. I've got a fix for this which should prevent people from making the same mistake in the future.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68064/new/

https://reviews.llvm.org/D68064





More information about the llvm-commits mailing list