drop unnecessary calls

The the scenario of creating signatures for a component, we always want
to respect the important flag if it is given. Therefore I think that we
can drop the `designSystem.compileAstNodes(candidate,
CompileAstFlags.None)` which saves us some time.

Also removed the `structuredClone` call. Whenever we handle `@apply`
which essentially happened the step before, we already start from a
cloned node internally. This call just meant we were doing it again.

Author: RobinMalfait

packages/tailwindcss/src/signatures.ts

@@ -44,17 +44,12 @@ export const computeUtilitySignature = new DefaultMap<
         // There's separate utility caches for respect important vs not
         // so we want to compile them both with `@theme inline` disabled
         for (let candidate of designSystem.parseCandidate(utility)) {
-          designSystem.compileAstNodes(candidate, CompileAstFlags.None)
           designSystem.compileAstNodes(candidate, CompileAstFlags.RespectImportant)
         }
 
         substituteAtApply(ast, designSystem)
       })
 
-      // We will be mutating the AST, so we need to clone it first to not affect
-      // the original AST
-      ast = structuredClone(ast)
-
       // Optimize the AST. This is needed such that any internal intermediate
       // nodes are gone. This will also cleanup declaration nodes with undefined
       // values or `--tw-sort` declarations.