r208722 - Add a requires for the arm-registered-target needed by this test as

Eric Christopher echristo at gmail.com
Tue May 13 13:21:51 PDT 2014


On Tue, May 13, 2014 at 1:20 PM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> On 13 May 2014 16:09, Eric Christopher <echristo at gmail.com> wrote:
>> Also as a note, driver tests shouldn't require registered targets if
>> they're just checking command lines. Also, no clang test should
>> require a registered target. I'm not sure why some do at the moment.
>
> This is a case where the -cc1 test requiring asm printing is somewhat
> reasonable. I don't think we have a better way to check that a codegen
> option passed to -cc1 is being used.
>

Right. We should only test that we've passed it. Make no claims about
it working :)

-eric




More information about the cfe-commits mailing list