phyBrackets added a comment. In D121045#3362373 <https://reviews.llvm.org/D121045#3362373>, @steakhal wrote:
> I would probably query the opcode only once and reuse it, but is also fine. > > Btw whats your intention making this change? Do you plan greater > refactorings/cleanups? Yeah, I think query the opcode once is a nice idea tho, let me do this in this patch only. And yeah I'm thinking of refactoring/cleanups where it needed and would love to make some big refactors. But I found this when I was looking into this issue (https://github.com/llvm/llvm-project/issues/53564 and needs to look at the deadstorechecker file.) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121045/new/ https://reviews.llvm.org/D121045 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
