Skip to content

Commit 1ef8f1f

Browse files
authored
[ctor-eval] Switch logging from stderr to stdout (#4432)
This logging is central to what this tool does, and not optional, so stdout makes more sense I think. Also, as I'm re-integrating this on the emscripten side, this makes it simpler.
1 parent ce8750b commit 1ef8f1f

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

src/tools/wasm-ctor-eval.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -500,7 +500,7 @@ void evalCtors(Module& wasm, std::vector<std::string> ctors) {
500500
// go one by one, in order, until we fail
501501
// TODO: if we knew priorities, we could reorder?
502502
for (auto& ctor : ctors) {
503-
std::cerr << "trying to eval " << ctor << '\n';
503+
std::cout << "trying to eval " << ctor << '\n';
504504
Export* ex = wasm.getExportOrNull(ctor);
505505
if (!ex) {
506506
Fatal() << "export not found: " << ctor;
@@ -510,10 +510,10 @@ void evalCtors(Module& wasm, std::vector<std::string> ctors) {
510510
} catch (FailToEvalException& fail) {
511511
// that's it, we failed, so stop here, cleaning up partial
512512
// memory changes first
513-
std::cerr << " ...stopping since could not eval: " << fail.why << "\n";
513+
std::cout << " ...stopping since could not eval: " << fail.why << "\n";
514514
return;
515515
}
516-
std::cerr << " ...success on " << ctor << ".\n";
516+
std::cout << " ...success on " << ctor << ".\n";
517517

518518
// Success, the entire function was evalled! Apply the results of
519519
// execution to the module.
@@ -528,7 +528,7 @@ void evalCtors(Module& wasm, std::vector<std::string> ctors) {
528528
}
529529
} catch (FailToEvalException& fail) {
530530
// that's it, we failed to even create the instance
531-
std::cerr << " ...stopping since could not create module instance: "
531+
std::cout << " ...stopping since could not create module instance: "
532532
<< fail.why << "\n";
533533
return;
534534
}
@@ -600,13 +600,13 @@ int main(int argc, const char* argv[]) {
600600

601601
{
602602
if (options.debug) {
603-
std::cerr << "reading...\n";
603+
std::cout << "reading...\n";
604604
}
605605
ModuleReader reader;
606606
try {
607607
reader.read(options.extra["infile"], wasm);
608608
} catch (ParseException& p) {
609-
p.dump(std::cerr);
609+
p.dump(std::cout);
610610
Fatal() << "error in parsing input";
611611
}
612612
}
@@ -639,7 +639,7 @@ int main(int argc, const char* argv[]) {
639639

640640
if (options.extra.count("output") > 0) {
641641
if (options.debug) {
642-
std::cerr << "writing..." << std::endl;
642+
std::cout << "writing..." << std::endl;
643643
}
644644
ModuleWriter writer;
645645
writer.setBinary(emitBinary);

0 commit comments

Comments
 (0)