From 5781ca340a2e9800a7e622eef8fffa72413d91cf Mon Sep 17 00:00:00 2001 From: Gurchetan Singh Date: Thu, 4 Jan 2024 17:17:09 -0800 Subject: [PATCH] genvk: options.filename --> args.target Previously, for gfxstream the filename was "CMakeLists.txt", but that was changed with the prior commit. Not having a filename doesn't affect codegen, but does throw an error afterwards when timing statistics are thrown out. Luckily, args.target and options.filename are more or less synonymous in genvk.py, so use that. This should be fairly upstreamable too if we choose to do so. Reviewed-by: Aaron Ruby Acked-by: Yonggang Luo Acked-by: Adam Jackson Part-of: --- src/gfxstream/codegen/scripts/genvk.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gfxstream/codegen/scripts/genvk.py b/src/gfxstream/codegen/scripts/genvk.py index f5f4c548d27..d4c1ab64d53 100755 --- a/src/gfxstream/codegen/scripts/genvk.py +++ b/src/gfxstream/codegen/scripts/genvk.py @@ -1038,7 +1038,7 @@ def genTarget(args): createGenerator = genOpts[args.target][0] options = genOpts[args.target][1] - logDiag('* Building', options.filename) + logDiag('* Building', args.target) logDiag('* options.versions =', options.versions) logDiag('* options.emitversions =', options.emitversions) logDiag('* options.defaultExtensions =', options.defaultExtensions) @@ -1234,7 +1234,7 @@ if __name__ == '__main__': else: startTimer(args.time) reg.apiGen() - endTimer(args.time, '* Time to generate ' + options.filename + ' =') + endTimer(args.time, '* Time to generate ' + args.target + ' =') if not args.quiet: - logDiag('* Generated', options.filename) + logDiag('* Generated', args.target)