(Apologies to Jeff who's getting this twice because I didn't hit
reply-all the first time.)

On 11/17/19 6:56 PM, Jeff Law wrote:

> While scanning this patch I did notice the introduction of
> CC_STATUS_INIT in output_{and,ior,xor}si.  You might want to check that.

That is intentional. CC_STATUS_INIT can be used without cc0, and there's
precedent in (IIRC) ARM. We need it to get final to tell us when it
encounters a label - those don't make it to the postscan_insn hook.

> So unless there's objections over the next say 48-72 hrs, let's get the
> kit in and we can iterate if there's further issues that need resolving.

Cool. I'll wait for a final confirmation.


Bernd

Reply via email to