wangpc-pp wrote: > I don't want to add this to clang just because there's no agreement on how to > derive ranges/mark derived ranges on llvm.read_register in instcombine. Have > we considered what that could look like?
Yes, I had considered to do it in instcombine. But `llvm.read_register` is not a target-specific intrinsic so it won't go through `XXXTargetTransformInfo::instCombineIntrinsic`, so we have to do it in the target-independent code but this is not encouraged according to https://llvm.org/docs/InstCombineContributorGuide.html#canonicalization-and-target-independence. https://github.com/llvm/llvm-project/pull/219924 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
