GitLab

The same name was used multiple times, because the counter was not increased anymore.
This was broken more than 6 years ago in 9d564d61.
This fixes opt/proc_cloning003.c.

This fixes opt/proc_cloning001.c.

clone_entity() sets the ld_name now.

This prevents triggering the unique name check in add_compound_member().
This partially fixes opt/proc_cloning001.c.

This fixes backend/convconv2.c.
This typo was introduced in afd4e3d6.

It checked for Const on the wrong side, so it always failed.
Then the code further down used the wrong accessor functions.

Switch lowering always gives an unsigned selector, so 'transform_zext()' is sufficient.

cleanup: Use 'new_global_entity()' instead of 'new_entity()' + 'set_entity_visibility()' + 'set_entity_linkage()'.

In one case the modes do not match: T -> M.
In the other case, a node is created just to set a transformed node for it.
In both cases the source nodes are not looked at anymore.

The verifier would have complained already, if a shift had float mode.

This fixes backend/shift2.c.

All callers use X86_SIZE_32.
Also any smaller size requires a third operand, because the result register does not get completely overwritten.

Now the original node name is set for all backends, not just ia32.
Further, no code in the backends is needed.
All new nodes, which are reachable from the returned transformed node and do not have a original node yet, automatically get assigned the original node.

Taking explicit size+po2align makes things easier compared to the
previously used ir_type: ir_type* is a middleend concept not everything
in the backend is represented as a type/mode anymore (x86_insn_size_t).
Furthermore we only respected size+alignment of that type anyway,
additional semantics would be unexpectedly ignored. It also simplifies
the code in some places.

This should also naturally fix a problem in 188.amp where we wrongly
passed the type for mode_Ls instead of the type for mode_D before.