New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Less RAM usage during Compilation of CTRE tests #389
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small tweaks todo
test/TokenTest.cpp
Outdated
} | ||
|
||
static constexpr auto x = cls(TurtleTokenCtre::PnCharsBaseString); | ||
// static constexpr auto x = cls(TurtleTokenCtre::PnCharsBaseString); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete out-commented code.
test/TokenTestCtreHelper.cpp
Outdated
// | ||
// Created by johannes on 04.05.21. | ||
// |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Proper Comment.
test/TokenTestCtreHelper.h
Outdated
static bool matchStringLiteralLongQuoteString(std::string_view s); | ||
static bool matchStringLiteralLongSingleQuoteString(std::string_view s); | ||
static bool matchIriref(std::string_view s); | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
newline
c8a7a28
to
9eed874
Compare
* This cleans up the software architecture. * Also equivalently refactor some of the TokenizerTests to use less RAM during compilation (The CTRE module is very RAM intensive)
9eed874
to
0c74314
Compare
The
TokenTest
required an insane amount of RAM to compile (4.5 GB), because the CTRE library is very expensive at compile time. This problem is mitigated in this PR by factoring out certain (duplicate) calls to CTRE to a separate compilation unit.