This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Eliminate location wrappers in tree_nop_conversion/STRIP_NOPS


On 12/19/2017 05:44 PM, David Malcolm wrote:
> On Tue, 2017-12-19 at 23:03 +0100, Jakub Jelinek wrote:
>> On Tue, Dec 19, 2017 at 04:59:34PM -0500, Jason Merrill wrote:
>>>> Or do you mean it should strip just the special VIEW_CONVERT_EXPR
>>>> that has type identical to the operand's type?
>>>
>>> That; interpreting something as the same type seems like a nop.
>>
>> Ok, that makes sense.
>>
>> 	Jakub
> 
> Thanks.
> 
> How does the following look?
> 
> Bootstrap&regrtest in progress; OK for trunk if it passes? (once the rest of
> the kit is approved, of course)
> 
> gcc/ChangeLog:
> 	* tree.c (tree_nop_conversion): Return true for location wrapper
> 	nodes.
> 	(selftest::check_strip_nops): New function.
> 	(selftest::test_location_wrappers): Verify that STRIP_NOPS removes
> 	wrappers.
OK.
jeff


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]