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

[Sergej Jaskiewicz via Phabricator via llvm-commits]

broadwaylamb updated this revision to Diff 218505.
broadwaylamb added a comment.

Use `nullptr` instead of `0`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67089

Files:
  llvm/unittests/ExecutionEngine/MCJIT/MCJITTestAPICommon.h


Index: llvm/unittests/ExecutionEngine/MCJIT/MCJITTestAPICommon.h
===================================================================
--- llvm/unittests/ExecutionEngine/MCJIT/MCJITTestAPICommon.h
+++ llvm/unittests/ExecutionEngine/MCJIT/MCJITTestAPICommon.h
@@ -55,7 +55,7 @@
 
   bool HostCanBeTargeted() {
     std::string Error;
-    return TargetRegistry::lookupTarget(HostTriple, Error) != 0;
+    return TargetRegistry::lookupTarget(HostTriple, Error) != nullptr;
   }
 
   /// Returns true if the host architecture is known to support MCJIT


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67089.218505.patch
Type: text/x-patch
Size: 547 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190903/37e7b1ec/attachment.bin>