On Thu, May 16, 2013 at 4:45 AM, Enea Zaffanella <[email protected]>wrote:
> On 05/15/2013 10:25 PM, Richard Smith wrote: > >> Please move the tests into test/CodeGenCXX, merge into existing test >> files, and ensure that -emit-llvm is on the RUN: line. Maybe >> cxx11-thread-local.cpp for the first test and anonymous-namespaces.cpp >> for the second one? >> >> Otherwise, LGTM, thanks! >> > > Added the patch and one of the tests in r181998. > > As for the other test, adding it at the end of cxx11-thread-local.cpp > results in a CHECK line failing to find its target. Since I know very > little about llvm bytecode, I left this addition to someone more > knowledgeable ... if you prefer, I can add those lines in their own test > file. Added in r182043.
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
