Add flag to reverse errors

forc-pkg/src/manifest.rs

@@ -223,6 +223,9 @@ pub struct BuildProfile {
     pub json_abi_with_callpaths: bool,
     #[serde(default)]
     pub error_on_warnings: bool,
+    #[serde(default)]
+    pub reverse_results: bool,
+    pub experimental_private_modules: bool,
 }
 
 impl Dependency {
@@ -666,6 +669,8 @@ impl BuildProfile {
             include_tests: false,
             json_abi_with_callpaths: false,
             error_on_warnings: false,
+            experimental_private_modules: false,
+            reverse_results: false,
         }
     }
 
@@ -682,6 +687,8 @@ impl BuildProfile {
             include_tests: false,
             json_abi_with_callpaths: false,
             error_on_warnings: false,
+            experimental_private_modules: false,
+            reverse_results: false,
         }
     }
 }

forc-pkg/src/pkg.rs

@@ -261,6 +261,8 @@ pub struct PrintOpts {
     pub intermediate_asm: bool,
     /// Print the generated Sway IR (Intermediate Representation).
     pub ir: bool,
+    /// Output build errors and warnings in reverse order.
+    pub reverse_order: bool,
 }
 
 #[derive(Default, Clone)]
@@ -1775,8 +1777,9 @@ pub fn compile(
         sway_build_config(pkg.manifest_file.dir(), &entry_path, pkg.target, profile)?
     );
     let terse_mode = profile.terse;
+    let reverse_results = profile.reverse_results;
     let fail = |warnings, errors| {
-        print_on_failure(terse_mode, warnings, errors);
+        print_on_failure(terse_mode, warnings, errors, reverse_results);
         bail!("Failed to compile {}", pkg.name);
     };
 
@@ -2258,7 +2261,7 @@ pub fn build(
         };
 
         let fail = |warnings, errors| {
-            print_on_failure(profile.terse, warnings, errors);
+            print_on_failure(profile.terse, warnings, errors, profile.reverse_results);
             bail!("Failed to compile {}", pkg.name);
         };
 
@@ -2346,7 +2349,10 @@ pub fn build(
             contract_id_value.clone(),
         ) {
             Ok(o) => o,
-            Err(errs) => return fail(&[], &errs),
+            Err(errs) => {
+                print_on_failure(profile.terse, &[], &errs, profile.reverse_results);
+                bail!("Failed to compile {}", pkg.name);
+            }
         };
 
         let mut compiled = compile(

forc-plugins/forc-client/src/op/deploy.rs

@@ -241,6 +241,7 @@ fn build_opts_from_cmd(cmd: &cmd::Deploy) -> pkg::BuildOpts {
             finalized_asm: cmd.print.finalized_asm,
             intermediate_asm: cmd.print.intermediate_asm,
             ir: cmd.print.ir,
+            reverse_order: cmd.print.reverse_order,
         },
         time_phases: cmd.print.time_phases,
         minify: pkg::MinifyOpts {

forc-plugins/forc-client/src/op/run.rs

@@ -162,6 +162,7 @@ fn build_opts_from_cmd(cmd: &cmd::Run) -> pkg::BuildOpts {
             finalized_asm: cmd.print.finalized_asm,
             intermediate_asm: cmd.print.intermediate_asm,
             ir: cmd.print.ir,
+            reverse_order: cmd.print.reverse_order,
         },
         minify: pkg::MinifyOpts {
             json_abi: cmd.minify.json_abi,

forc-util/src/lib.rs

@@ -378,13 +378,23 @@ pub fn print_warnings(
     ));
 }
 
-pub fn print_on_failure(terse_mode: bool, warnings: &[CompileWarning], errors: &[CompileError]) {
+pub fn print_on_failure(
+    terse_mode: bool,
+    warnings: &[CompileWarning],
+    errors: &[CompileError],
+    reverse_results: bool,
+) {
     let e_len = errors.len();
     let w_len = warnings.len();
 
     if !terse_mode {
-        warnings.iter().for_each(format_warning);
-        errors.iter().for_each(format_err);
+        if reverse_results {
+            warnings.iter().rev().for_each(format_warning);
+            errors.iter().rev().for_each(format_err);
+        } else {
+            warnings.iter().for_each(format_warning);
+            errors.iter().for_each(format_err);
+        }
     }
 
     if e_len == 0 && w_len > 0 {

forc/src/cli/commands/test.rs

@@ -200,6 +200,7 @@ fn opts_from_cmd(cmd: Command) -> forc_test::Opts {
             finalized_asm: cmd.build.print.finalized_asm,
             intermediate_asm: cmd.build.print.intermediate_asm,
             ir: cmd.build.print.ir,
+            reverse_order: cmd.build.print.reverse_order,
         },
         time_phases: cmd.build.print.time_phases,
         minify: pkg::MinifyOpts {

forc/src/cli/shared.rs

@@ -85,6 +85,9 @@ pub struct Print {
     /// Output the time elapsed over each part of the compilation process.
     #[clap(long)]
     pub time_phases: bool,
+    /// Output build errors and warnings in reverse order.
+    #[clap(long)]
+    pub reverse_order: bool,
 }
 
 /// Package-related options.

forc/src/ops/forc_build.rs

@@ -25,6 +25,7 @@ fn opts_from_cmd(cmd: BuildCommand) -> pkg::BuildOpts {
             finalized_asm: cmd.build.print.finalized_asm,
             intermediate_asm: cmd.build.print.intermediate_asm,
             ir: cmd.build.print.ir,
+            reverse_order: cmd.build.print.reverse_order,
         },
         time_phases: cmd.build.print.time_phases,
         minify: pkg::MinifyOpts {

forc/src/ops/forc_contract_id.rs

@@ -61,6 +61,7 @@ fn build_opts_from_cmd(cmd: &ContractIdCommand) -> pkg::BuildOpts {
             finalized_asm: cmd.print.finalized_asm,
             intermediate_asm: cmd.print.intermediate_asm,
             ir: cmd.print.ir,
+            reverse_order: cmd.print.reverse_order,
         },
         time_phases: cmd.print.time_phases,
         minify: pkg::MinifyOpts {

forc/src/ops/forc_predicate_root.rs

@@ -29,6 +29,7 @@ fn build_opts_from_cmd(cmd: PredicateRootCommand) -> pkg::BuildOpts {
             finalized_asm: cmd.print.finalized_asm,
             intermediate_asm: cmd.print.intermediate_asm,
             ir: cmd.print.ir,
+            reverse_order: cmd.print.reverse_order,
         },
         time_phases: cmd.print.time_phases,
         minify: pkg::MinifyOpts {