Keep disambiguating number in expanded node name

[GMNGeoffrey]

Nodes get named like add, add_1, etc to disambiguate ones with the same name. But the node naming in expansion throws away this suffix. I think it's helpful to leave it in, when viewing the IR. Nodes don't have to have unique names, but it's nice if they do.

I just updated one lit test here as an example because I want to confirm that this change is desirable before doing the work of updating the rest.

DO_NOT_COMMIT: I haven't updated all the lit tests

skip-ci: I know that some of the lit tests are failing. Want to get agreement before updating them.

[GMNGeoffrey]

Huh, it doesn't seem like the skip-ci trailer works in this repo. Note that the turbine contributing guidelines point to the IREE contributing guidelines, which document the ci control options. You should perhaps consider adding them to turbine. Whoever came up with them created a really cool and useful feature 😜 <-- @ScottTodd