Skip to content

Commit d8f8562

Browse files
authored
[RISC-V] Support displaying GTF_ADDRMODE_NO_CSE flag on GT_CAST (#125516)
Follow-up to #122088. Add support to display the `GTF_ADDRMODE_NO_CSE` flag on `GT_CAST`. Before ``` N024 ( 2, 5) [000389] -----+---U- | | +--* CAST long <- uint $386 ``` After ``` N024 ( 2, 5) [000389] -----+-N-U- | | +--* CAST long <- uint $386 ``` Part of #84834, cc @dotnet/samsung
1 parent dea4756 commit d8f8562

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

src/coreclr/jit/gentree.cpp

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -11586,20 +11586,11 @@ void Compiler::gtDispNode(GenTree* tree, IndentStack* indentStack, _In_ _In_opt_
1158611586
/* Then print the general purpose flags */
1158711587
GenTreeFlags flags = tree->gtFlags;
1158811588

11589-
if (tree->OperIsBinary() || tree->OperIsMultiOp())
11589+
if (tree->IsPartOfAddressMode())
1159011590
{
11591-
genTreeOps oper = tree->OperGet();
11592-
11593-
// Check for GTF_ADDRMODE_NO_CSE flag on add/mul/shl Binary Operators
11594-
if ((oper == GT_ADD) || (oper == GT_MUL) || (oper == GT_LSH))
11595-
{
11596-
if ((tree->gtFlags & GTF_ADDRMODE_NO_CSE) != 0)
11597-
{
11598-
flags |= GTF_DONT_CSE; // Force the GTF_ADDRMODE_NO_CSE flag to print out like GTF_DONT_CSE
11599-
}
11600-
}
11591+
flags |= GTF_DONT_CSE; // Force the GTF_ADDRMODE_NO_CSE flag to print out like GTF_DONT_CSE
1160111592
}
11602-
else // !(tree->OperIsBinary() || tree->OperIsMultiOp())
11593+
if (!(tree->OperIsBinary() || tree->OperIsMultiOp()))
1160311594
{
1160411595
// the GTF_REVERSE flag only applies to binary operations (which some MultiOp nodes are).
1160511596
flags &= ~GTF_REVERSE_OPS;

0 commit comments

Comments
 (0)