Make Tuple2 Lookupable

[mwachs5]

Contributor Checklist

• [N/A] Did you add Scaladoc to every public function/method?
• Did you add at least one test demonstrating the PR?
• Did you delete any extraneous printlns/debugging code?
• Did you specify the type of improvement?
• [N/A] Did you add appropriate documentation in docs/src?
• Did you state the API impact?
• Did you specify the code generation impact?
• Did you request a desired merge strategy?
• Did you add text to be included in the Release Notes for this change?

Type of Improvement

• new feature/API

API Impact

Adds Tuple2 as something which is Lookupable (can be marked @public).

Backend Code Generation Impact

It doesnt change any verilog.

Desired Merge Strategy

• Squash: The PR will be squashed and merged (choose this if you have no preference.

Release Notes

Make Tuple2 Lookupable

Reviewer Checklist (only modified by reviewer)

• Did you add the appropriate labels?
• Did you mark the proper milestone (Bug fix: 3.4.x, [small] API extension: 3.5.x, API modification or big change: 3.6.0)?
• Did you review?
• Did you check whether all relevant Contributor checkboxes have been checked?
• Did you mark as Please Merge?

[mwachs5]

I made this a draft to discuss how to extend past a tuple 2... is it really just a copy paste for each one up to tupleN?

[mwachs5]

This is failing scalafmtCheck, but when I run sbt scalafmt on my own machine there is no diff. Any suggestions?

[debs-sifive]

@mwachs5 make sure you're using the same scala version as CI when calling scalafmt. I ran into this issue and fixed it by using

sbt ++2.12.15 scalafmtAll

[jackkoenig]

sbt ++2.12.15 scalafmtAll

For the record, I don't think the scala version matters but it's very import to run scalafmtAll instead of just scalafmt. The former includes files in src/test/scala.

[jackkoenig]

Do you want to merge this as is or would you like to add some more Tuples?

[mwachs5]

Do you want to merge this as is or would you like to add some more Tuples?

I think I would want to merge this as is because it allows Maps to be implemented...