[PATCH] D33243: [Atomics][LoopIdiom] Recognize unordered atomic memcpy
Anna Thomas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 6 10:57:02 PDT 2017
anna added a comment.
FYI: The committed patch (https://reviews.llvm.org/rL304806)caused failures in multiple targets (passed on X86, PPC and s390), since the unordered-atomic-memcpy.ll test was in the common LoopIdiom directory.
Discussing with echristo on IRC: Specifying target-triple as X86 is not enough. It would still be run on targets where the X86 backend is not compiled in.
I've checked in the fix (https://reviews.llvm.org/rL304809), which is to move the file to the X86 specific subdirectory, where the lit.cfg chooses that the test should not be run on any other target.
Repository:
rL LLVM
https://reviews.llvm.org/D33243
More information about the llvm-commits
mailing list