[PATCH] D52550: [ARM] Check for sel intrinsic use in ARM CGP

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 26 11:03:28 PDT 2018


efriedma added a comment.

This isn't really correct, in any useful sense... we can't assume the current translation unit reflects the entire program.

I mean, if you really think this optimization is sufficiently useful, we can add a flag for it.  Presumably user code could enable it if they want to, but it would have to be disabled for libraries shipped with the compiler (like compiler-rt).


https://reviews.llvm.org/D52550





More information about the llvm-commits mailing list