[cfe-commits] r168958 - in /cfe/trunk/lib: Basic/Targets.cpp Driver/Tools.cpp

Chandler Carruth chandlerc at google.com
Thu Nov 29 16:50:50 PST 2012


On Thu, Nov 29, 2012 at 4:43 PM, Quentin Colombet <qcolombet at apple.com> wrote:
>
> On Nov 29, 2012, at 4:38 PM, Chandler Carruth <chandlerc at google.com> wrote:
>
>> On Thu, Nov 29, 2012 at 4:35 PM, Quentin Colombet <qcolombet at apple.com> wrote:
>>> Hi Chandler,
>>>
>>> Here is a patch with a test case.
>>>
>>> I didn't put a test case in the original patch as Evan suggested me to remove it for the related commit on the llvm side.
>>
>> Sure. I usually just submit the LLVM bits first, and then do both
>> sides of the clang change (and the tests) in one commit. ::shrug::
>
> Good to know, thanks.
>
> Is the test case looking good for you?

Oh, yes. Sorry, that was meant to be implied with the sure. =D



More information about the cfe-commits mailing list