https://github.com/agozillon commented:
Recent PR update aims to address the prior comments where they're still relevant after making the suggested changes by @skatrak to move away from using the DenseIntElementsAttr (thank you very much for the suggestion, it's simplified things a fair bit in a few places :-)). Please do have a subsequent look over the changeset to check any comments made previously that were still relevant after the move to a new attribute type for the indices have been addressed (quite a few comments and some may have been lost in the shuffle/changes so sorry if I have missed any, please feel free to re-iterate on them with a ping and I'll address them or comment as to why I couldn't)! https://github.com/llvm/llvm-project/pull/96266 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
