[PATCH] D86952: [MIRVRegNamer] MachineInstr StableHashing.

Puyan Lotfi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 3 10:06:12 PDT 2020


plotfi added a comment.

In D86952#2254859 <https://reviews.llvm.org/D86952#2254859>, @paquette wrote:

> Is there any way to test the hash itself on certain instructions?
>
> e.g. verify that two instructions with different address spaces get a different hash?

I think most of the tests where I added the -mir-vreg-namer-use-stable-hash  line are checking for that. For instance the `; CHECK-NEXT: %bb0_{{[0-9]+}}__1:` lines pretty much ensure that the `__1` is there, which means there was not more than one collision.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86952/new/

https://reviews.llvm.org/D86952



More information about the llvm-commits mailing list