[llvm] Revisit advanced LLJIT examples and tests (PR #76236)
Stefan Gränitz via llvm-commits
llvm-commits at lists.llvm.org
Fri Dec 22 07:43:25 PST 2023
================
@@ -121,10 +159,11 @@ int main(int argc, char *argv[]) {
int InFD = 0;
int OutFD = 0;
+ std::vector<StringRef> TestOutputFlags;
+
if (argc < 2)
printErrorAndExit("insufficient arguments");
else {
----------------
weliveindetail wrote:
`printErrorAndExit()` is noreturn. Is there a good reason to keep this `else` branch? Otherwise I'd drop it and move the `TestOutputFlags` closer to the loop below. Also, I still need to update the help output.
https://github.com/llvm/llvm-project/pull/76236
More information about the llvm-commits
mailing list