[PATCH] D74228: [PatternMatch] Match XOR variant of unsigned-add overflow check.

[via llvm-commits]

Attached reproducer:

-------------- next part --------------
A non-text attachment was scrubbed...
Name: sm.repro.tar.gz
Type: application/x-gzip
Size: 992248 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200219/eb58cd44/attachment-0001.bin>
-------------- next part --------------


> On Feb 19, 2020, at 10:38 AM, Florian Hahn via Phabricator <reviews at reviews.llvm.org> wrote:
> 
> fhahn added a comment.
> 
> In D74228#1883057 <https://reviews.llvm.org/D74228#1883057>, @vsk wrote:
> 
>> Not yet 100% sure, but I suspect this is breaking the clang self-host, see:
>> 
>>  FAILED: tools/clang/lib/Basic/CMakeFiles/obj.clangBasic.dir/SourceManager.cpp.o 
>>  /Users/buildslave/jenkins/workspace/lldb-cmake/host-compiler/bin/clang++  -DGTEST_HAS_RTTI=0 -D_DEBUG -D_GNU_SOURCE -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D__STDC_LIMIT_MACROS -Itools/clang/lib/Basic -I/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/clang/lib/Basic -I/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/clang/include -Itools/clang/include -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include/libxml2 -Iinclude -I/Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include -Wdocumentation -fPIC -fvisibility-inlines-hidden -Werror=date-time -Werror=unguarded-availability-new -fmodules -fmodules-cache-path=/Users/buildslave/jenkins/workspace/lldb-cmake/lldb-build/module.cache -fcxx-modules -Wall -Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual -Wmissing-field-initializers -pedantic -Wno-long-long -Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type -Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wstring-conversion -fdiagnostics-color -fno-common -Woverloaded-virtual -Wno-nested-anon-types -O3  -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk    -fno-exceptions -fno-rtti -UNDEBUG -std=c++14 -MD -MT tools/clang/lib/Basic/CMakeFiles/obj.clangBasic.dir/SourceManager.cpp.o -MF tools/clang/lib/Basic/CMakeFiles/obj.clangBasic.dir/SourceManager.cpp.o.d -o tools/clang/lib/Basic/CMakeFiles/obj.clangBasic.dir/SourceManager.cpp.o -c /Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/clang/lib/Basic/SourceManager.cpp
>>  Instruction does not dominate all uses!
>>    %1 = load i32, i32* %Size.i, align 8, !tbaa !66
>>    %0 = call { i32, i1 } @llvm.uadd.with.overflow.i32(i32 %FID.coerce, i32 %1)
>>  in function _ZNK5clang13SourceManager17getPreviousFileIDENS_6FileIDE
>>  fatal error: error in backend: Broken function found, compilation aborted!
>> 
>> 
>> http://lab.llvm.org:8080/green/view/LLDB/job/lldb-cmake/9597
> 
> 
> Probably. I've revert the change for now.
> 
> 
> Repository:
>  rG LLVM Github Monorepo
> 
> CHANGES SINCE LAST ACTION
>  https://reviews.llvm.org/D74228/new/
> 
> https://reviews.llvm.org/D74228
> 
> 
>