fhahn added a comment.

In D116919#3236899 <https://reviews.llvm.org/D116919#3236899>, @mstorsjo wrote:

> The relanded version in rG41fbdfa4d5601cccbcdc0ded8ef35190d502f7f3 
> <https://reviews.llvm.org/rG41fbdfa4d5601cccbcdc0ded8ef35190d502f7f3> seems 
> to be breaking some builds with PCH for me, failing asserts like this:
>
>   clang: ../tools/clang/include/clang/Basic/SourceLocation.h:135: 
> clang::SourceLocation 
> clang::SourceLocation::getLocWithOffset(clang::SourceLocation::IntTy) const: 
> Assertion `((getOffset()+Offset) & MacroIDBit) == 0 && "offset overflow"' 
> failed.

This also breaks stage2 builds with debug info, e.g. 
https://green.lab.llvm.org/green/job/clang-stage2-Rthinlto/5088/console

Given that the bot has been red a while now, I reverted the commit eadb4cfeeff5 
<https://reviews.llvm.org/rGeadb4cfeeff55678eac0c9225f095567e1564424> for now.

The crash should be reproducible when doing a stage2 build with debug info. It 
crashes, e.g. when building llvm-project/llvm/lib/Support/Timer.cpp

   
/Users/buildslave/jenkins/workspace/clang-stage2-Rthinlto/host-compiler/bin/clang++
  -DGTEST_HAS_RTTI=0 -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS 
-D__STDC_LIMIT_MACROS -Ilib/Support 
-I/Users/buildslave/jenkins/workspace/clang-stage2-Rthinlto/llvm-project/llvm/lib/Support
 -Iinclude 
-I/Users/buildslave/jenkins/workspace/clang-stage2-Rthinlto/llvm-project/llvm/include
 -fno-stack-protector -fno-common -Wno-profile-instr-unprofiled -fPIC 
-fvisibility-inlines-hidden -Werror=date-time 
-Werror=unguarded-availability-new -fmodules 
-fmodules-cache-path=/Users/buildslave/jenkins/workspace/clang-stage2-Rthinlto/clang-build/Build/module.cache
 -fcxx-modules -Xclang -fmodules-local-submodule-visibility -gmodules -Wall 
-Wextra -Wno-unused-parameter -Wwrite-strings -Wcast-qual 
-Wmissing-field-initializers -pedantic -Wno-long-long -Wc++98-compat-extra-semi 
-Wimplicit-fallthrough -Wcovered-switch-default -Wno-noexcept-type 
-Wnon-virtual-dtor -Wdelete-non-virtual-dtor -Wsuggest-override 
-Wstring-conversion -Wmisleading-indentation -fdiagnostics-color -flto=thin  
-O2 -g -DNDEBUG -isysroot 
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk
   -std=c++14  -fno-exceptions -fno-rtti -MD -MT 
lib/Support/CMakeFiles/LLVMSupport.dir/Timer.cpp.o -MF 
lib/Support/CMakeFiles/LLVMSupport.dir/Timer.cpp.o.d -o 
lib/Support/CMakeFiles/LLVMSupport.dir/Timer.cpp.o -c 
/Users/buildslave/jenkins/workspace/clang-stage2-Rthinlto/llvm-project/llvm/lib/Support/Timer.cpp
  Assertion failed: (((getOffset()+Offset) & MacroIDBit) == 0 && "offset 
overflow"), function getLocWithOffset, file 
/Users/buildslave/jenkins/workspace/clang-stage1-RA/llvm-project/clang/include/clang/Basic/SourceLocation.h,
 line 135.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116919/new/

https://reviews.llvm.org/D116919

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to