[llvm] [clang-tools-extra] [clang] [compiler-rt] [InferAddressSpaces] Fix constant replace to avoid modifying other functions (PR #70611)

Wenju He via cfe-commits cfe-commits at lists.llvm.org
Sun Nov 12 18:14:55 PST 2023


wenju-he wrote:

> I think it would be better if we could eliminate ConstantExpr addrspacecasts from the IR altogether, which would avoid most of the complexity here. I would also somewhat prefer to push this DFS into a helper function, but can live with it inline as-is

thank you for the review. I'll draft another PR to modify convertUsersOfConstantsToInstructions to allow change in a function only, so that DFS is not need here.

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


More information about the cfe-commits mailing list