[PATCH] D21739: [TLI] Add functions determining if int parameters/returns should be zeroext/signext.

Marcin Koƛcielnicki via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 21 11:41:03 PST 2016


koriakin added a comment.

In https://reviews.llvm.org/D21739#601531, @alekseyshl wrote:

> It seems like this revision causes failures on sanitizer-x86_64-linux-bootstrap, at least check-clang ubsan is unhappy:
>
> /mnt/b/sanitizer-buildbot2/sanitizer-x86_64-linux-bootstrap/build/llvm/lib/Analysis/TargetLibraryInfo.cpp:453:59: runtime error: load of value 32, which is not a valid value for type 'bool'
>
> The actual loaded value varies.


Right, my bad - I didn't notice the initialize() function has an early return for AMDGPU targets and thus initialization of the new fields may not be reached.  I'll just move this code to the beginning of the function, it should fix the failures.


Repository:
  rL LLVM

https://reviews.llvm.org/D21739





More information about the llvm-commits mailing list