llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) <details> <summary>Changes</summary> It's unnecessary and broke a builder: https://lab.llvm.org/buildbot/#/builders/228/builds/3735 --- Full diff: https://github.com/llvm/llvm-project/pull/207480.diff 1 Files Affected: - (modified) clang/lib/AST/ByteCode/Interp.cpp (-7) ``````````diff diff --git a/clang/lib/AST/ByteCode/Interp.cpp b/clang/lib/AST/ByteCode/Interp.cpp index dc93cc4fd698c..b099a3e8571f7 100644 --- a/clang/lib/AST/ByteCode/Interp.cpp +++ b/clang/lib/AST/ByteCode/Interp.cpp @@ -3231,13 +3231,6 @@ PRESERVE_NONE static bool BCP(InterpState &S, CodePtr OpPC, int32_t Offset, // We have already evaluated this speculation's EndSpeculation opcode. assert(S.SpeculationDepth == DepthBefore - 1); - // Jump to end label. This is a little tricker than just RealPC += Offset - // because our usual jump instructions don't have any arguments, to the offset - // we get is a little too much and we need to subtract the size of the - // bool and PrimType arguments again. - int32_t ParamSize = align(sizeof(PrimType)); - assert(Offset >= ParamSize); - return true; } `````````` </details> https://github.com/llvm/llvm-project/pull/207480 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
