[llvm] [TableGen] More generically handle tied source operands in CompressInstEmitter. (PR #146183)

Rahul Joshi via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 2 06:09:24 PDT 2025


jurahul wrote:

LGTM as well as it seems the remaining 2 comments are addressed. I just had one more minor nit. In

```
StringMap<std::pair<unsigned, unsigned>>
```

Should we use a struct with members named DagOpNo and MIOpNo so that in the code it's obvious what we are using instead of pair.first/.second?

https://github.com/llvm/llvm-project/pull/146183


More information about the llvm-commits mailing list