[PATCH] D124526: [SimpleLoopUnswitch] Collect either logical ANDs/ORs but not both.

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

Hello I think that since this patch there has been a failure on the PowerPC big endian bootstrap build. I had originally thought it was D123801 <https://reviews.llvm.org/D123801> because it had the words "powerpc" and "bigendian" in it.

After being able to reproduce it, I think it's from this one though. This example through -simple-loop-unswitch fails now: https://godbolt.org/z/fjWd443P3
It's quite heavily reduced. If it does not reproduce, the original came from running:

  bin/clang++ -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Ilib/Target/WebAssembly -I../llvm/lib/Target/WebAssembly -Iinclude -I../llvm/include -fPIC -fno-semantic-interposition -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wc++98-compat-extra-semi -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override -Wno-comment -Wstring-conversion -Wmisleading-indentation -fdiagnostics-color -ffunction-sections -fdata-sections -O3 -DNDEBUG -fvisibility=hidden -fno-exceptions -fno-rtti -UNDEBUG -std=c++14 -c ../llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp --target=powerpc64-linux-gnu -I/usr/powerpc-linux-gnu/include/c++/7/powerpc-linux-gnu/64


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124526