Re: [PATCH][AArch64] Set TREE_TARGET_GLOBALS in aarch64_set_current_function when new tree is the default node to recalculate optab availability

2016-02-29 Thread Christophe Lyon
On 29 February 2016 at 15:28, Kyrill Tkachov wrote: > Hi Crhistophe, > > > On 29/02/16 14:10, Christophe Lyon wrote: >> >> On 26 February 2016 at 16:51, James Greenhalgh >> wrote: >>> >>> On Thu, Feb 25, 2016 at 11:04:21AM +, Kyrill

Re: [PATCH][AArch64] Set TREE_TARGET_GLOBALS in aarch64_set_current_function when new tree is the default node to recalculate optab availability

2016-02-29 Thread Kyrill Tkachov
Hi Crhistophe, On 29/02/16 14:10, Christophe Lyon wrote: On 26 February 2016 at 16:51, James Greenhalgh wrote: On Thu, Feb 25, 2016 at 11:04:21AM +, Kyrill Tkachov wrote: Hi all, Seems like aarch64 is suffering from something similar to PR 69245 as well. If a

Re: [PATCH][AArch64] Set TREE_TARGET_GLOBALS in aarch64_set_current_function when new tree is the default node to recalculate optab availability

2016-02-29 Thread Christophe Lyon
On 26 February 2016 at 16:51, James Greenhalgh wrote: > On Thu, Feb 25, 2016 at 11:04:21AM +, Kyrill Tkachov wrote: >> Hi all, >> >> Seems like aarch64 is suffering from something similar to PR 69245 as well. >> If a target pragma sets the target state to the same as

Re: [PATCH][AArch64] Set TREE_TARGET_GLOBALS in aarch64_set_current_function when new tree is the default node to recalculate optab availability

2016-02-26 Thread James Greenhalgh
On Thu, Feb 25, 2016 at 11:04:21AM +, Kyrill Tkachov wrote: > Hi all, > > Seems like aarch64 is suffering from something similar to PR 69245 as well. > If a target pragma sets the target state to the same as the > target_option_default_node the node is just a pointer to >

[PATCH][AArch64] Set TREE_TARGET_GLOBALS in aarch64_set_current_function when new tree is the default node to recalculate optab availability

2016-02-25 Thread Kyrill Tkachov
Hi all, Seems like aarch64 is suffering from something similar to PR 69245 as well. If a target pragma sets the target state to the same as the target_option_default_node the node is just a pointer to target_option_default_node rather than a distinct identical node. So we must still restore