Re: [PATCH] D24087: [lit] Allow more file extensions for test cases.

2016-09-06 Thread Eric Fiselier via cfe-commits
EricWF added inline comments. Comment at: test/libcxx/test/format.py:69 @@ +68,3 @@ +name_root, name_ext = os.path.splitext(name) +is_sh_test = name_root.endswith('.sh') +is_pass_test = name_root.endswith('.pass') logan wrote: > EricWF wrot

Re: [PATCH] D24087: [lit] Allow more file extensions for test cases.

2016-09-06 Thread Logan Chien via cfe-commits
logan added inline comments. Comment at: test/libcxx/test/format.py:69 @@ +68,3 @@ +name_root, name_ext = os.path.splitext(name) +is_sh_test = name_root.endswith('.sh') +is_pass_test = name_root.endswith('.pass') EricWF wrote: > Since we on

Re: [PATCH] D24087: [lit] Allow more file extensions for test cases.

2016-09-03 Thread Eric Fiselier via cfe-commits
EricWF accepted this revision. EricWF added a comment. This revision is now accepted and ready to land. Woops I see the test case now. LGTM. Comment at: test/libcxx/test/format.py:69 @@ +68,3 @@ +name_root, name_ext = os.path.splitext(name) +is_sh_test = name_ro

Re: [PATCH] D24087: [lit] Allow more file extensions for test cases.

2016-09-03 Thread Eric Fiselier via cfe-commits
EricWF added a comment. I don't think assembly based `.pass` and `.fail` tests will work quite yet. The `CXXCompiler` object used by the test format explicitly passes `-xc++ %s` in some cases. I'm assuming you already have a libc++abi assembly test? Could you put it up for review? https://re