[llvm] r317626 - Fix build bots after r317622

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 7 15:01:31 PST 2017


On Tue, Nov 7, 2017 at 2:37 PM, Craig Topper via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> This wil always return 0 in a release build. Is that what you want? Or
> should it be calling lookup in release builds?
>

This is definitely not right. The right fix here is to hoist the
lookup function outside of the assertion to make sure it
doesn't get stripped in builds without assertions.
Thanks for reverting.

--
Davide


More information about the llvm-commits mailing list