Can we get a testcase for this bug? Seems like we can't have any coverage
for the case where IsArmArch is supposed to be false.

On 3 Jun 2017 9:47 am, "Galina Kistanova via cfe-commits" <
cfe-commits@lists.llvm.org> wrote:

Author: gkistanova
Date: Sat Jun  3 11:47:06 2017
New Revision: 304663

URL: http://llvm.org/viewvc/llvm-project?rev=304663&view=rev
Log:
Fixed warning: enum constant in boolean context.

Modified:
    cfe/trunk/lib/Driver/ToolChains/Linux.cpp

Modified: cfe/trunk/lib/Driver/ToolChains/Linux.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/
ToolChains/Linux.cpp?rev=304663&r1=304662&r2=304663&view=diff
============================================================
==================
--- cfe/trunk/lib/Driver/ToolChains/Linux.cpp (original)
+++ cfe/trunk/lib/Driver/ToolChains/Linux.cpp Sat Jun  3 11:47:06 2017
@@ -822,8 +822,9 @@ SanitizerMask Linux::getSupportedSanitiz
   const bool IsAArch64 = getTriple().getArch() == llvm::Triple::aarch64 ||
                          getTriple().getArch() == llvm::Triple::aarch64_be;
   const bool IsArmArch = getTriple().getArch() == llvm::Triple::arm ||
-                         llvm::Triple::thumb || llvm::Triple::armeb ||
-                         llvm::Triple::thumbeb;
+                         getTriple().getArch() == llvm::Triple::thumb ||
+                         getTriple().getArch() == llvm::Triple::armeb ||
+                         getTriple().getArch() == llvm::Triple::thumbeb;
   SanitizerMask Res = ToolChain::getSupportedSanitizers();
   Res |= SanitizerKind::Address;
   Res |= SanitizerKind::Fuzzer;


_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to