LGTM. Discussed this with Sid offline and tests will be added as part of adding the encodings.

LGTM. Jyotsna looked at the patch and she is okay with the Hexagon changes as well.

Line 137 - remove "Is Dwarf number correct comment" and on the following lines as well.

Jyotsna - these patches look good. Go ahead and commit the remaining patches that add encoding bits to Hexagon after testing.

Thanks for identifying the bug and providing a fix, Tim. We'll take a look at the Hexagon bugfix.

Okay, please remember to add tests when you add v5 support. It need not be an elaborate set though that would be even better :)). We should have tests to ensure that the timing classes are actually consumed and used in codegen.

LGTM.

Test cases: Can you capture the change in timing classes as a scheduling change that can be tested?