[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