Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[mono] Make mono_inst_name less misleading #91042

Merged
merged 11 commits into from
Apr 9, 2024
Merged

[mono] Make mono_inst_name less misleading #91042

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
f459650
Wraping both declaration and definition in #ifndef DISABLE_LOGGING.
sayeedkhannabil Aug 24, 2023
65d9c26
Merge branch 'dotnet:main' into disableLogin_mono_inst_name
sayeedkhannabil Aug 28, 2023
0c381e1
Removed repeated #ifndef DISABLE_LOGIN from `helpers.c`. fixes #83545
sayeedkhannabil Aug 28, 2023
aca933e
Adding DISABLE_LOGGING for testing
sayeedkhannabil Sep 12, 2023
3bbbe75
Added DISABLE_LOOGING wrapper on all instances of mono_inst_name.
sayeedkhannabil Sep 12, 2023
b0c438f
Adjust based on feedback
steveisok Apr 1, 2024
77c6a2f
Merge remote-tracking branch 'upstream/main' into disableLogin_mono_i…
steveisok Apr 1, 2024
331643a
Merge remote-tracking branch 'origin/disableLogin_mono_inst_name' int…
steveisok Apr 1, 2024
8c96e92
Fix bad merge
steveisok Apr 1, 2024
c3701da
Merge remote-tracking branch 'upstream/main' into disableLogin_mono_i…
steveisok Apr 5, 2024
a09dc98
Fix more usages of mono_inst_name without M_PRI_INST
akoeplinger Apr 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions src/mono/mono/mini/helpers.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,7 @@ static const gint16 opidx [] = {
#endif
//#define ARCH_PREFIX "powerpc64-linux-gnu-"

#ifndef DISABLE_LOGGING
ivanpovazan marked this conversation as resolved.
Show resolved Hide resolved
const char*
mono_inst_name (int op) {
#ifndef DISABLE_LOGGING
Expand All @@ -90,6 +91,7 @@ mono_inst_name (int op) {
g_assert_not_reached ();
#endif
}
#endif

void
mono_blockset_print (MonoCompile *cfg, MonoBitSet *set, const char *name, guint idom)
Expand Down
2 changes: 2 additions & 0 deletions src/mono/mono/mini/mini.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2186,7 +2186,9 @@ GString *mono_print_ins_index_strbuf (int i, MonoInst *ins);
void mono_print_ins (MonoInst *ins);
void mono_print_bb (MonoBasicBlock *bb, const char *msg);
void mono_print_code (MonoCompile *cfg, const char *msg);
#ifndef DISABLE_LOGGING
const char* mono_inst_name (int op);
#endif
int mono_op_to_op_imm (int opcode);
int mono_op_imm_to_op (int opcode);
int mono_load_membase_to_load_mem (int opcode);
Expand Down