From f124e75656dd926d10e805f802e57eb9f7f6d4c5 Mon Sep 17 00:00:00 2001 From: Guozhi Wei Date: Tue, 5 Mar 2019 18:54:34 +0000 Subject: [PATCH] [X86] In X86DomainReassignment.cpp add enclosed registers to EnclosedEdges The variable X86DomainReassignment::EnclosedEdges is used to store registers that have been enclosed in some closure, so those registers will be ignored when create new closures. But there is no registers has ever been put into this set, so a single register can be enclosed in multiple closures, it significantly increase compile time. This patch adds a register into EnclosedEdges when it is enclosed into a closure. Differential Revision: https://reviews.llvm.org/D58646 llvm-svn: 355430 --- llvm/lib/Target/X86/X86DomainReassignment.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/llvm/lib/Target/X86/X86DomainReassignment.cpp b/llvm/lib/Target/X86/X86DomainReassignment.cpp index 86d9de22775d..b4bd63dad8ca 100644 --- a/llvm/lib/Target/X86/X86DomainReassignment.cpp +++ b/llvm/lib/Target/X86/X86DomainReassignment.cpp @@ -556,6 +556,7 @@ void X86DomainReassignment::buildClosure(Closure &C, unsigned Reg) { // Register already in this closure. if (!C.insertEdge(CurReg)) continue; + EnclosedEdges.insert(Reg); MachineInstr *DefMI = MRI->getVRegDef(CurReg); encloseInstr(C, DefMI);