On 2/3/2022 1:00 AM, Robin Dapp wrote:
Do you need to adjust anything now that this is emitting into TEMP
rather than TARGET?
The idea now is to emit to TEMP in the first pass and check if we read
the initial condition. Overwriting the condition (and of course reading
it in every sequence) is the reason temporaries were needed in the first
place.
In the second pass (always done now) we use TEMP for those
SETs/sequences before the last one that read the condition and TARGET
for those after it. Ideally it's TARGET for all of them which would
result in the best costing situation.
OK. Thanks for clarifying.
The whole logic was/is really a bit convoluted for what we want to
achieve but I'd argue it's not much worse than before ;)
Yea. And it's not that far off from things we do elsewhere -- we
generate some sequence then start grubbing around inside for things of
interest.
OK for the trunk.
Any thoughts on the ARC failure which is triggered by your patch and
still unresolved?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=104154
Jeff
