[llvm] [TableGen][GISel] Fix importing frameindex node (PR #120921)
Sergei Barannikov via llvm-commits
llvm-commits at lists.llvm.org
Sun Dec 22 23:06:24 PST 2024
================
@@ -1248,10 +1244,6 @@ Expected<action_iterator> GlobalISelEmitter::importExplicitUseRenderer(
DstMIBuilder.addRenderer<CopyRenderer>(Dst.getName());
return InsertPt;
}
- if (Dst.getOperator()->getName() == "tframeindex") {
----------------
s-barannikov wrote:
OK, considering there are other `t*` nodes, I'll just remove the check and make that and add `CopyRenderer` unconditionally. There is no good reason to treat `tframeindex` differently from other `t*` nodes, and whitelisting them isn't a scalable approach.
https://github.com/llvm/llvm-project/pull/120921
More information about the llvm-commits
mailing list