[llvm-commits] CVS: llvm/test/Transforms/InstCombine/APInt/

[Chris Lattner]

>> Hrm?  Why a subdirectory?
>
> Three reasons:
> 1. Many of the tests are similar but just modified for APInt. So  
> instead
> of having
>    sext-apint.ll and zext-apint.ll and or-apint.ll, etc., I just
> factored the name
>    out into a directory. Also reduces clutter in the
> test/Transforms/InstCombine
>    directory.

the is no logical difference between apint tests and non-apint tests  
for instcombine.  If you want to include apint in the filename,  
that's fine, but I don't think a directory is useful.

> 2. To allow "make check TESTSUITE=test/Transforms/InstCombine/ 
> APInt" to
> just check
>    the APInt versions of these tests.

This is a temporary convenience while you're working on apint'ifying  
instcombine.  This will not be useful once you're done.

-Chris