[PATCH] D67089: Skip MCJIT unit tests if LLVM is not configured for native compilation

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 3 11:32:30 PDT 2019


dblaikie added inline comments.


================
Comment at: llvm/unittests/ExecutionEngine/MCJIT/MCJITTestAPICommon.h:58
+    std::string Error;
+    return TargetRegistry::lookupTarget(HostTriple, Error) != 0;
+  }
----------------
Please use 'nullptr' rather than '0' for a null pointer constant.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67089





More information about the llvm-commits mailing list