Skip to content

Fix misleading log message in OR_MODE_BITS branch of CompareRunPlanData()#128

Merged
rrivera747 merged 2 commits intorrivera/betterCFORunPlanPrintoutCommentsfrom
copilot/sub-pr-125-another-one
Mar 9, 2026
Merged

Fix misleading log message in OR_MODE_BITS branch of CompareRunPlanData()#128
rrivera747 merged 2 commits intorrivera/betterCFORunPlanPrintoutCommentsfrom
copilot/sub-pr-125-another-one

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 9, 2026

The OR_MODE_BITS opcode branch in CompareRunPlanData() was emitting a trace message labelled "Found AND line", making it impossible to distinguish AND vs OR opcodes in diagnostic output.

Change

  • cfoInterfaceLib/CFO_Registers.cpp: Corrected log string from "Found AND line" to "Found OR line" in the OR_MODE_BITS block.
// Before
__COUTT__ << "Found AND line = " << l / 8 << " --> 0x" << std::hex << orMasks->get().back() << __E__;

// After
__COUTT__ << "Found OR line = " << l / 8 << " --> 0x" << std::hex << orMasks->get().back() << __E__;

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Address feedback on adding MARKER index to printouts for opcodes tracking Fix misleading log message in OR_MODE_BITS branch of CompareRunPlanData() Mar 9, 2026
@rrivera747 rrivera747 marked this pull request as ready for review March 9, 2026 17:53
@rrivera747 rrivera747 merged commit 9d63a5d into rrivera/betterCFORunPlanPrintoutComments Mar 9, 2026
@rrivera747 rrivera747 deleted the copilot/sub-pr-125-another-one branch March 9, 2026 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants