[cfe-commits] Please review: Hexagon V5 (floating point) support in cfe

Sirish Pande spande at codeaurora.org
Mon Apr 30 19:27:30 PDT 2012


On 4/30/2012 6:56 PM, Douglas Gregor wrote:
> On Apr 25, 2012, at 10:15 PM, Sirish Pande<spande at codeaurora.org>  wrote:
>
>> Hi,
>>
>> This is the support for floating point in Hexagon V5 architecture in cfe.
>>
>> This patch does not give warnings, and passes all tests with make check-all.
> Patch looks generally fine, but could you split it into two parts before committing? The first part would be the interesting changes (target description string, defines, driver stuff) and the second part would be the massive churn in BuiltinsHexagon.def/CGBuiltin.cpp
>
> 	- Doug

Sure can do - will split into two parts (basically intrinsics and the 
rest), and commit.

Sirish

-- 
Qualcomm Innovation Center, Inc is a member of Code Aurora Forum




More information about the cfe-commits mailing list