Omr: Rename NonHelper to Intrinsic?

[0dvictor]

In the discussion of #2997 and #3049, as well as development of atomic operation support, I found the name of "NonHelper" is quite confusing. They look alike to helper methods but not a real helper because CodeGen is expected to generate inlined sequences instead of a helper call. Similar concept is known as "intrinsic" or "intrinsic method/function" in many other compilers.
Should we change the concept of "NonHelper" to "Intrinsic", or some other name that less confusing?

[0dvictor]

For example change the enum CommonNonhelperSymbol to CommonIntrinsicSymbol.

[0dvictor]

@0xdaryl @andrewcraik @fjeremic @Leonardo2718 @mstoodle @ymanton

[andrewcraik]

I think a question such as this deserves to go on the agenda of one of the architecture meetings. @0xdaryl

[fjeremic]

I agree the term "NonHelper" is confusing. I'm in favor of renaming to "Intrinsic". I also agree we should bring this up on the architecture meeting discussing the topic.

[0xdaryl]

I'd like to move this discussion to today's architecture meeting: @0dvictor @ymanton @fjeremic @andrewcraik @gita-omr @Leonardo2718 @vijaysun-omr