[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