Skip to content

Add case GT_NOT: in optVNBasedFoldCurStmt next to case GT_NEG#123690

Closed
Copilot wants to merge 4 commits intomainfrom
copilot/add-case-gt-not-optvnfold
Closed

Add case GT_NOT: in optVNBasedFoldCurStmt next to case GT_NEG#123690
Copilot wants to merge 4 commits intomainfrom
copilot/add-case-gt-not-optvnfold

Conversation

Copy link
Contributor

Copilot AI commented Jan 27, 2026

Description

Enables VN-based folding for GT_NOT (bitwise NOT) in optVNBasedFoldCurStmt, matching the existing handling for GT_NEG.

Both are unary operators (GTK_UNOP) using GenTreeOp, and are already grouped together elsewhere in the JIT (e.g., OperIsRMWMemOp).

         case GT_NEG:
+        case GT_NOT:
         case GT_CAST:
Original prompt

Add case GT_NOT: in optVNBasedFoldCurStmt next to case GT_NEG


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: EgorBo <523221+EgorBo@users.noreply.github.com>
Copilot AI changed the title [WIP] Add case GT_NOT in optVNBasedFoldCurStmt next to case GT_NEG Add case GT_NOT: in optVNBasedFoldCurStmt next to case GT_NEG Jan 27, 2026
Copilot AI requested a review from EgorBo January 27, 2026 23:09
@EgorBo
Copy link
Member

EgorBo commented Jan 28, 2026

Will include in my other PR

@EgorBo EgorBo closed this Jan 28, 2026
@jkotas jkotas deleted the copilot/add-case-gt-not-optvnfold branch February 2, 2026 23:14
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.

2 participants