[PATCH] D142803: [ComplexLogicCombine 1/?] Implement a general way to simplify complex logical operations.

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D142803#4137888 <https://reviews.llvm.org/D142803#4137888>, @bcl5980 wrote:

>> I was curious if enabling this for all logic ops would cause any compile-time regressions, but it seems like it has almost no cost:
>> https://llvm-compile-time-tracker.com/compare.php?from=0e90cd7551f2d0b151f7406e8f3848ec54e650bf&to=ae505cb2a674ac4c240c94a74fc04ee274321697&stat=instructions:u
>
> Based on the result, can I just enable it for all logical operations by default and remove the option?

Yes, let's make the transform more general (and the patch becomes simpler if we remove the option).


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

https://reviews.llvm.org/D142803