Merge pull request #1897 from certik/print_unsigned

certik · web-flow · commit 21a7be2f1bd0 · 2023-06-13T07:43:54.000-06:00
Print unsigned ints correctly
diff --git a/src/libasr/codegen/asr_to_llvm.cpp b/src/libasr/codegen/asr_to_llvm.cpp
@@ -7195,19 +7195,19 @@ class ASRToLLVMVisitor : public ASR::BaseVisitor<ASRToLLVMVisitor>
             } else if (ASRUtils::is_unsigned_integer(*t)) {
                 switch( a_kind ) {
                     case 1 : {
-                        fmt.push_back("%hhi");
+                        fmt.push_back("%hhu");
                         break;
                     }
                     case 2 : {
-                        fmt.push_back("%hi");
+                        fmt.push_back("%hu");
                         break;
                     }
                     case 4 : {
-                        fmt.push_back("%d");
+                        fmt.push_back("%u");
                         break;
                     }
                     case 8 : {
-                        fmt.push_back("%lld");
+                        fmt.push_back("%llu");
                         break;
                     }
                     default: {

Original file line number	Diff line number	Diff line change
`@@ -7195,19 +7195,19 @@ class ASRToLLVMVisitor : public ASR::BaseVisitor<ASRToLLVMVisitor>`
`7195`	`7195`	`} else if (ASRUtils::is_unsigned_integer(*t)) {`
`7196`	`7196`	`switch( a_kind ) {`
`7197`	`7197`	`case 1 : {`
`7198`		`- fmt.push_back("%hhi");`
	`7198`	`+ fmt.push_back("%hhu");`
`7199`	`7199`	`break;`
`7200`	`7200`	`}`
`7201`	`7201`	`case 2 : {`
`7202`		`- fmt.push_back("%hi");`
	`7202`	`+ fmt.push_back("%hu");`
`7203`	`7203`	`break;`
`7204`	`7204`	`}`
`7205`	`7205`	`case 4 : {`
`7206`		`- fmt.push_back("%d");`
	`7206`	`+ fmt.push_back("%u");`
`7207`	`7207`	`break;`
`7208`	`7208`	`}`
`7209`	`7209`	`case 8 : {`
`7210`		`- fmt.push_back("%lld");`
	`7210`	`+ fmt.push_back("%llu");`
`7211`	`7211`	`break;`
`7212`	`7212`	`}`
`7213`	`7213`	`default: {`