Don't dump an uninitialized bitset in LSRA.
authorPat Gavlin <pagavlin@microsoft.com>
Thu, 8 Jun 2017 19:00:45 +0000 (12:00 -0700)
committerPat Gavlin <pagavlin@microsoft.com>
Thu, 8 Jun 2017 19:00:45 +0000 (12:00 -0700)
LSRA was unconditionally dumping `fpCalleeSaveCandidateVars`, which is
not initialized if we are not enregistering any lclVars. This was
causing an assertion with dumping enabled.

src/jit/lsra.cpp

index 7cbfa51..3372e27 100644 (file)
@@ -2341,7 +2341,7 @@ void LinearScan::identifyCandidates()
     if (VERBOSE)
     {
         printf("\nFP callee save candidate vars: ");
-        if (!VarSetOps::IsEmpty(compiler, fpCalleeSaveCandidateVars))
+        if (enregisterLocalVars && !VarSetOps::IsEmpty(compiler, fpCalleeSaveCandidateVars))
         {
             dumpConvertedVarSet(compiler, fpCalleeSaveCandidateVars);
             printf("\n");