[sdcc-devel] [ sdcc-Feature Requests-3526285 ] Non-destructive and for ifx (hc08)

[SourceForge.net <no...@so...>]

Feature Requests item #3526285, was opened at 2012-05-13 04:02
Message generated for change (Comment added) made by epetrich
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=350599&aid=3526285&group_id=599

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: hc08 port
Group: None
Status: Open
Priority: 5
Private: No
Submitted By: Philipp Klaus Krause (spth)
Assigned to: Erik Petrich (epetrich)
Summary: Non-destructive and for ifx (hc08)

Initial Comment:
And that is only used for a following ifx should use the bit and brset instructions where doing so provides and advantage.

Philipp


----------------------------------------------------------------------

>Comment By: Erik Petrich (epetrich)
Date: 2012-05-15 11:18

Message:
brset/brclr only support direct addressing mode, not extended. It works
when reg has the __data qualifier. (but I also found a bug in my
implementation that I fixed in #7708)

----------------------------------------------------------------------

Comment By: Philipp Klaus Krause (spth)
Date: 2012-05-15 06:14

Message:
I still see the old

	lda	_galois_lfsr_reg_1_2
	and	#0x01
	beq	00103$

generated for the testcase I tried #7701 on:

unsigned char galois_lfsr(void)
{
	static unsigned char reg = 1;
	reg = ((reg & 1) ? (reg >> 1)^0xe1 : (reg >> 1));
	return(reg);
}


----------------------------------------------------------------------

Comment By: Erik Petrich (epetrich)
Date: 2012-05-15 03:16

Message:
The brset/brclr half of this feature request has been implemented in #7701.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=350599&aid=3526285&group_id=599