[llvm] Port Preserved[Module|Function]HashAnalysis to Analysis (PR #116108)

Justin Fargnoli via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 14 09:42:40 PST 2024


justinfargnoli wrote:

Thank you for the clarification @nikic! 

My original thought process was that `IRNormalizer` should not make any semantic changes to the IR and thus should not invalidate any analyses besides the hashes. 

However, it seems like the correct thought process is to be conservative and invalidate everything except CFG analyses because an analysis may depend on the syntactic structure of the IR. Am I on the right track here? 

To address you're feedback I've added cd500d28cba3177c213f2f2faf50f14ea56e230b to the `IRNormalizer` PR and will close this one. 

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


More information about the llvm-commits mailing list