[PATCH] D106334: Initialize common options in `getRegisteredOptions`

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 1 20:53:48 PDT 2021


MaskRay added inline comments.


================
Comment at: llvm/unittests/Support/CommandLineInit/CMakeLists.txt:1
+set(test_name CommandLineInitTests)
+set(test_suite UnitTests)
----------------
This file looks a bit complex. Is all stuff here needed?


================
Comment at: llvm/unittests/Support/CommandLineInit/CommandLineInitTest.cpp:31
+
+#if defined(_WIN32)
+  // Disable all of the possible ways Windows conspires to make automated
----------------
State where this piece of code is copied from.

unittest/UnitTestMain/TestMain.cpp ?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D106334/new/

https://reviews.llvm.org/D106334



More information about the llvm-commits mailing list