[llvm] [Kaleidoscope] Fix newlines in listings to match the tutorial text. (PR #87922)
Kirill Podoprigora via llvm-commits
llvm-commits at lists.llvm.org
Sun Apr 7 05:58:45 PDT 2024
https://github.com/Eclips4 created https://github.com/llvm/llvm-project/pull/87922
Output in tutorial looks like that `\n` is should be there:
https://github.com/llvm/llvm-project/blob/7a4e89761a13bfad27a2614ecea5e8698f50336c/llvm/docs/tutorial/MyFirstLanguageFrontend/LangImpl03.rst?plain=1#L447-L458
Actual output with current code (no newline):
```llvm
ready> def foo(a b) a*a + 2*a*b + b*b;
ready> Read function definition:define double @foo(double %a, double %b) {
entry:
%multmp = fmul double %a, %a
%multmp1 = fmul double 2.000000e+00, %a
%multmp2 = fmul double %multmp1, %b
%addtmp = fadd double %multmp, %multmp2
%multmp3 = fmul double %b, %b
%addtmp4 = fadd double %addtmp, %multmp3
ret double %addtmp4
}
```
The same applies for `extern`.
>From 2995a2223696b3b0d59abf95fba3e77d900044ba Mon Sep 17 00:00:00 2001
From: Kirill Podoprigora <kirill.bast9 at mail.ru>
Date: Sun, 7 Apr 2024 15:50:13 +0300
Subject: [PATCH] Fix newlines to match the output in tutorial
---
llvm/examples/Kaleidoscope/BuildingAJIT/Chapter1/toy.cpp | 4 ++--
llvm/examples/Kaleidoscope/BuildingAJIT/Chapter2/toy.cpp | 4 ++--
llvm/examples/Kaleidoscope/BuildingAJIT/Chapter3/toy.cpp | 4 ++--
llvm/examples/Kaleidoscope/BuildingAJIT/Chapter4/toy.cpp | 2 +-
llvm/examples/Kaleidoscope/Chapter3/toy.cpp | 6 +++---
llvm/examples/Kaleidoscope/Chapter4/toy.cpp | 4 ++--
llvm/examples/Kaleidoscope/Chapter5/toy.cpp | 4 ++--
llvm/examples/Kaleidoscope/Chapter6/toy.cpp | 4 ++--
llvm/examples/Kaleidoscope/Chapter7/toy.cpp | 4 ++--
llvm/examples/Kaleidoscope/Chapter8/toy.cpp | 2 +-
10 files changed, 19 insertions(+), 19 deletions(-)
diff --git a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter1/toy.cpp b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter1/toy.cpp
index 1b35ba404d29b2..327ecfc0f65bf6 100644
--- a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter1/toy.cpp
+++ b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter1/toy.cpp
@@ -1115,7 +1115,7 @@ static void InitializeModule() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
auto TSM = ThreadSafeModule(std::move(TheModule), std::move(TheContext));
@@ -1131,7 +1131,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter2/toy.cpp b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter2/toy.cpp
index 1b35ba404d29b2..327ecfc0f65bf6 100644
--- a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter2/toy.cpp
+++ b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter2/toy.cpp
@@ -1115,7 +1115,7 @@ static void InitializeModule() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
auto TSM = ThreadSafeModule(std::move(TheModule), std::move(TheContext));
@@ -1131,7 +1131,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter3/toy.cpp b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter3/toy.cpp
index 1b35ba404d29b2..327ecfc0f65bf6 100644
--- a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter3/toy.cpp
+++ b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter3/toy.cpp
@@ -1115,7 +1115,7 @@ static void InitializeModule() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
auto TSM = ThreadSafeModule(std::move(TheModule), std::move(TheContext));
@@ -1131,7 +1131,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter4/toy.cpp b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter4/toy.cpp
index 2c8d4941291e08..f454c8f395a36a 100644
--- a/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter4/toy.cpp
+++ b/llvm/examples/Kaleidoscope/BuildingAJIT/Chapter4/toy.cpp
@@ -1129,7 +1129,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/Chapter3/toy.cpp b/llvm/examples/Kaleidoscope/Chapter3/toy.cpp
index 03563006685add..65ecf446fe8477 100644
--- a/llvm/examples/Kaleidoscope/Chapter3/toy.cpp
+++ b/llvm/examples/Kaleidoscope/Chapter3/toy.cpp
@@ -530,7 +530,7 @@ static void InitializeModule() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
}
@@ -543,7 +543,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
}
@@ -557,7 +557,7 @@ static void HandleTopLevelExpression() {
// Evaluate a top-level expression into an anonymous function.
if (auto FnAST = ParseTopLevelExpr()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read top-level expression:");
+ fprintf(stderr, "Read top-level expression:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
diff --git a/llvm/examples/Kaleidoscope/Chapter4/toy.cpp b/llvm/examples/Kaleidoscope/Chapter4/toy.cpp
index 1bbc294bf35263..f232c19095338e 100644
--- a/llvm/examples/Kaleidoscope/Chapter4/toy.cpp
+++ b/llvm/examples/Kaleidoscope/Chapter4/toy.cpp
@@ -599,7 +599,7 @@ static void InitializeModuleAndManagers() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
ExitOnErr(TheJIT->addModule(
@@ -615,7 +615,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/Chapter5/toy.cpp b/llvm/examples/Kaleidoscope/Chapter5/toy.cpp
index 48936bddb1d4f2..eb54a96f44f954 100644
--- a/llvm/examples/Kaleidoscope/Chapter5/toy.cpp
+++ b/llvm/examples/Kaleidoscope/Chapter5/toy.cpp
@@ -873,7 +873,7 @@ static void InitializeModuleAndManagers() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
ExitOnErr(TheJIT->addModule(
@@ -889,7 +889,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/Chapter6/toy.cpp b/llvm/examples/Kaleidoscope/Chapter6/toy.cpp
index ebe4322287b21f..7894c8523401fc 100644
--- a/llvm/examples/Kaleidoscope/Chapter6/toy.cpp
+++ b/llvm/examples/Kaleidoscope/Chapter6/toy.cpp
@@ -992,7 +992,7 @@ static void InitializeModuleAndManagers() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
ExitOnErr(TheJIT->addModule(
@@ -1008,7 +1008,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/Chapter7/toy.cpp b/llvm/examples/Kaleidoscope/Chapter7/toy.cpp
index 68208c4f3394ab..a34b16c9d830c5 100644
--- a/llvm/examples/Kaleidoscope/Chapter7/toy.cpp
+++ b/llvm/examples/Kaleidoscope/Chapter7/toy.cpp
@@ -1161,7 +1161,7 @@ static void InitializeModuleAndManagers() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
ExitOnErr(TheJIT->addModule(
@@ -1177,7 +1177,7 @@ static void HandleDefinition() {
static void HandleExtern() {
if (auto ProtoAST = ParseExtern()) {
if (auto *FnIR = ProtoAST->codegen()) {
- fprintf(stderr, "Read extern: ");
+ fprintf(stderr, "Read extern:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
FunctionProtos[ProtoAST->getName()] = std::move(ProtoAST);
diff --git a/llvm/examples/Kaleidoscope/Chapter8/toy.cpp b/llvm/examples/Kaleidoscope/Chapter8/toy.cpp
index ae2f9c7059e5fb..f0935653327f3a 100644
--- a/llvm/examples/Kaleidoscope/Chapter8/toy.cpp
+++ b/llvm/examples/Kaleidoscope/Chapter8/toy.cpp
@@ -1118,7 +1118,7 @@ static void InitializeModuleAndPassManager() {
static void HandleDefinition() {
if (auto FnAST = ParseDefinition()) {
if (auto *FnIR = FnAST->codegen()) {
- fprintf(stderr, "Read function definition:");
+ fprintf(stderr, "Read function definition:\n");
FnIR->print(errs());
fprintf(stderr, "\n");
}
More information about the llvm-commits
mailing list