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