[PATCH] D15118: [InstCombine] Make MatchBSwap also match bit reversals

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 2 08:15:27 PST 2015


Hi David,

Sure, will do. This change has no measurable time impact on a bootstrap
build of clang.

James
On Wed, 2 Dec 2015 at 16:05, David Majnemer via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> majnemer added a comment.
>
> Do you have data on how this change impacts compile time?
>
>
> ================
> Comment at: lib/Transforms/InstCombine/InstCombineAndOrXor.cpp:20
> @@ -19,2 +19,3 @@
>  #include "llvm/Transforms/Utils/CmpInstAnalysis.h"
> +#include "llvm/ADT/StringExtras.h"
>  using namespace llvm;
> ----------------
> Please sort this include.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D15118
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151202/95f2e4c2/attachment.html>


More information about the llvm-commits mailing list