fix: 5 dead-code warnings in main.c (expression generates no code) #18

New issue

Closed

opened 2026-07-27 00:29:22 +00:00 by time · 3 comments

time commented

2026-07-27 00:29:22 +00:00

Contributor

XC8 reports 5 expression-generates-no-code warnings at lines 171, 185, 340, 350, 359 in main.c. These are standalone side-effect-free expressions that should either be removed or have a comment explaining why they are there.

daru commented

2026-07-27 00:33:10 +00:00

Owner

Those lines should do something, but they are being optimized out. Not sure why pin_write calls would be getting dropped, though.

Those lines *should* do something, but they are being optimized out. Not sure why pin_write calls would be getting dropped, though.

time referenced this issue from a pull request that will close it,

2026-07-27 01:03:16 +00:00

fix: pin_write calls optimized away by XC8 #19

time commented

2026-07-27 01:03:16 +00:00

Author

Contributor

Fixed in PR #19: replaced ternary pin_write macro with direct LATAx = val writes to XC8's volatile __bit variables, which the compiler cannot optimize away.

Fixed in PR #19: replaced ternary `pin_write` macro with direct `LATAx = val` writes to XC8's `volatile __bit` variables, which the compiler cannot optimize away.

daru commented

2026-07-27 01:03:48 +00:00

Owner

Verified

daru closed this issue

2026-07-27 01:03:48 +00:00