[ARM] OK to delete test/CodeGen/a5.c and test/CodeGen/a15.c ?

Alp Toker alp at nuanti.com
Thu Nov 21 21:40:27 PST 2013


Sure Artyom, go ahead and please remove test/CodeGen/r5.c as well while 
you're at it.

With those three gone, I see only two other tests remaining in CodeGen 
that don't check the output in any way:

   test/CodeGen/mips-clobber-reg.c: Looks like a Frontend test?
   test/CodeGen/attribute-section-data-common.c: No idea what this is.

Alp.


On 21/11/2013 16:44, Artyom Skrobov wrote:
> Hello,
>
> These two tests are apparently there with the sole purpose of ensuring clang
> accepts -mcpu=cortex-a5 and -mcpu=cortex-a15
>
> Those same options are tested, among many other things, in
> test/Preprocessor/arm-target-features.c and test/Driver/arm-cortex-cpus.c,
> so the two aforementioned tests seem totally redundant.
>
> Does anyone mind me deleting these two tests?
>
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list