[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