[PATCH] R600: Handle -mcpu option

Tom Stellard tom at stellard.net
Thu Mar 14 08:10:04 PDT 2013


On Wed, Mar 13, 2013 at 11:17:40PM +0200, Dmitri Gribenko wrote:
> On Wed, Mar 13, 2013 at 11:08 PM, Tom Stellard <tom at stellard.net> wrote:
> > On Tue, Mar 12, 2013 at 10:45:27PM +0200, Dmitri Gribenko wrote:
> >> On Tue, Mar 12, 2013 at 10:21 PM, Tom Stellard <tom at stellard.net> wrote:
> >> >  lib/Driver/Tools.cpp |   19 +++++++++++++++++++
> >> >  lib/Driver/Tools.h   |    1 +
> >>
> >> Hi Tom,
> >>
> >> Is it possible to add tests?
> >>
> >
> > Sure, here is an updated patch with a test case.
> 
> You should FileCheck against the generated command lines directly.
> You need to invoke clang with -### argument for that.  See
> test/Driver/mips-features.c for an example.
>

Hi Dmitri,

Here is an updated patch that uses the -### option for tests.

-Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-R600-Handle-mcpu-option-v3.patch
Type: text/x-diff
Size: 7373 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130314/08f0db10/attachment.patch>


More information about the cfe-commits mailing list