From e78735752024d73655e8538e165e735e19c41903 Mon Sep 17 00:00:00 2001 From: Dan Gohman Date: Tue, 20 Mar 2018 15:22:22 -0700 Subject: [PATCH] Wrap build.py's contents in a main function. This prevents its local variables from becoming global variables. --- lib/cretonne/meta/build.py | 34 +++++++++++++++++++++------------- 1 file changed, 21 insertions(+), 13 deletions(-) diff --git a/lib/cretonne/meta/build.py b/lib/cretonne/meta/build.py index a94b9ce706..8c83300604 100644 --- a/lib/cretonne/meta/build.py +++ b/lib/cretonne/meta/build.py @@ -14,19 +14,27 @@ import gen_legalizer import gen_registers import gen_binemit -parser = argparse.ArgumentParser(description='Generate sources for Cretonne.') -parser.add_argument('--out-dir', help='set output directory') -args = parser.parse_args() -out_dir = args.out_dir +def main(): + # type: () -> None + parser = argparse.ArgumentParser( + description='Generate sources for Cretonne.') + parser.add_argument('--out-dir', help='set output directory') -isas = isa.all_isas() + args = parser.parse_args() + out_dir = args.out_dir -gen_types.generate(out_dir) -gen_instr.generate(isas, out_dir) -gen_settings.generate(isas, out_dir) -gen_encoding.generate(isas, out_dir) -gen_legalizer.generate(isas, out_dir) -gen_registers.generate(isas, out_dir) -gen_binemit.generate(isas, out_dir) -gen_build_deps.generate() + isas = isa.all_isas() + + gen_types.generate(out_dir) + gen_instr.generate(isas, out_dir) + gen_settings.generate(isas, out_dir) + gen_encoding.generate(isas, out_dir) + gen_legalizer.generate(isas, out_dir) + gen_registers.generate(isas, out_dir) + gen_binemit.generate(isas, out_dir) + gen_build_deps.generate() + + +if __name__ == "__main__": + main()