check_starlark_stack_size and resolve_package_listing concurrently

Summary:
This should have no effect practically because `check_starlark_stack_size` is cache hit except first time.

But I need to fetch profile mode there, and that should be done concurrently with `resolve_package_listing`, so I'll add another `compute` branch.

Reviewed By: JakobDegen

Differential Revision: D58789803

fbshipit-source-id: 6b5d0e77390a49446d314790b2c38544bbbd6b39

Author: stepancheg

app/buck2_interpreter_for_build/src/interpreter/dice_calculation_delegate.rs

@@ -414,15 +414,15 @@ impl<'c, 'd: 'c> DiceCalculationDelegate<'c, 'd> {
     }
 
     async fn resolve_package_listing(
-        &mut self,
+        ctx: &mut DiceComputations<'_>,
         package: PackageLabel,
     ) -> anyhow::Result<PackageListing> {
         span_async(
             buck2_data::LoadPackageStart {
                 path: package.as_cell_path().to_string(),
             },
             async {
-                let result = DicePackageListingResolver(self.ctx)
+                let result = DicePackageListingResolver(ctx)
                     .resolve_package_listing(package.dupe())
                     .await;
                 (
@@ -441,9 +441,13 @@ impl<'c, 'd: 'c> DiceCalculationDelegate<'c, 'd> {
         package: PackageLabel,
         profiler_instrumentation: &mut StarlarkProfilerOpt<'_>,
     ) -> buck2_error::Result<Arc<EvaluationResult>> {
-        check_starlark_stack_size(self.ctx).await?;
-
-        let listing = self.resolve_package_listing(package.dupe()).await?;
+        let ((), listing) = self
+            .ctx
+            .try_compute2(
+                |ctx| check_starlark_stack_size(ctx).boxed(),
+                |ctx| Self::resolve_package_listing(ctx, package.dupe()).boxed(),
+            )
+            .await?;
 
         let build_file_path = BuildFilePath::new(package.dupe(), listing.buildfile().to_owned());
         let (ast, deps) = self