Oops, my fault for reading commits in chronological order. Thanks, Tim!
On Feb 7, 2013, at 9:17 , Jordan Rose <[email protected]> wrote: > Actually, while you're here it would be nice to change this to a > FileCheck-based test... > > > On Feb 7, 2013, at 7:11 , Tim Northover <[email protected]> wrote: > >> Author: tnorthover >> Date: Thu Feb 7 09:11:48 2013 >> New Revision: 174621 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=174621&view=rev >> Log: >> Only check for volatile memcpys in test. >> >> AArch64 handles aggFct's return struct slightly differently, leading >> to an extra memcpy. This test is fortunately only concerned about >> volatile copies, so we can modify the grep text to filter it. >> >> Modified: >> cfe/trunk/test/CodeGen/volatile.c >> >> Modified: cfe/trunk/test/CodeGen/volatile.c >> URL: >> http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGen/volatile.c?rev=174621&r1=174620&r2=174621&view=diff >> ============================================================================== >> --- cfe/trunk/test/CodeGen/volatile.c (original) >> +++ cfe/trunk/test/CodeGen/volatile.c Thu Feb 7 09:11:48 2013 >> @@ -1,6 +1,6 @@ >> // RUN: %clang_cc1 -emit-llvm < %s -o %t >> // RUN: grep volatile %t | count 28 >> -// RUN: grep memcpy %t | count 7 >> +// RUN: grep "memcpy.*, i1 true" %t | count 6 >> >> // The number 28 comes from the current codegen for volatile loads; >> // if this number changes, it's not necessarily something wrong, but >> >> >> _______________________________________________ >> cfe-commits mailing list >> [email protected] >> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
