[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