[PATCH] D66681: [clang-doc] Bump BitcodeWriter max line number to 32U
This revision was automatically updated to reflect the committed changes. Closed by commit rL369811: [clang-doc] Bump BitcodeWriter max line number to 32U (authored by juliehockett, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D66681?vs=216942=216952#toc Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66681/new/ https://reviews.llvm.org/D66681 Files: clang-tools-extra/trunk/clang-doc/BitcodeWriter.h Index: clang-tools-extra/trunk/clang-doc/BitcodeWriter.h === --- clang-tools-extra/trunk/clang-doc/BitcodeWriter.h +++ clang-tools-extra/trunk/clang-doc/BitcodeWriter.h @@ -40,7 +40,7 @@ static constexpr unsigned IntSize = 16U; static constexpr unsigned StringLengthSize = 16U; static constexpr unsigned FilenameLengthSize = 16U; - static constexpr unsigned LineNumberSize = 16U; + static constexpr unsigned LineNumberSize = 32U; static constexpr unsigned ReferenceTypeSize = 8U; static constexpr unsigned USRLengthSize = 6U; static constexpr unsigned USRBitLengthSize = 8U; Index: clang-tools-extra/trunk/clang-doc/BitcodeWriter.h === --- clang-tools-extra/trunk/clang-doc/BitcodeWriter.h +++ clang-tools-extra/trunk/clang-doc/BitcodeWriter.h @@ -40,7 +40,7 @@ static constexpr unsigned IntSize = 16U; static constexpr unsigned StringLengthSize = 16U; static constexpr unsigned FilenameLengthSize = 16U; - static constexpr unsigned LineNumberSize = 16U; + static constexpr unsigned LineNumberSize = 32U; static constexpr unsigned ReferenceTypeSize = 8U; static constexpr unsigned USRLengthSize = 6U; static constexpr unsigned USRBitLengthSize = 8U; ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D66681: [clang-doc] Bump BitcodeWriter max line number to 32U
phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM (small nit, people usually use PRXXX rather than bXXX to refer to issues) CHANGES SINCE LAST ACTION https://reviews.llvm.org/D66681/new/ https://reviews.llvm.org/D66681 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[PATCH] D66681: [clang-doc] Bump BitcodeWriter max line number to 32U
juliehockett created this revision. juliehockett added a reviewer: phosek. juliehockett added a project: clang-tools-extra. b43039 reports hitting the assert on a very large file, so bumping this to allow for larger files. https://reviews.llvm.org/D66681 Files: clang-tools-extra/clang-doc/BitcodeWriter.h Index: clang-tools-extra/clang-doc/BitcodeWriter.h === --- clang-tools-extra/clang-doc/BitcodeWriter.h +++ clang-tools-extra/clang-doc/BitcodeWriter.h @@ -40,7 +40,7 @@ static constexpr unsigned IntSize = 16U; static constexpr unsigned StringLengthSize = 16U; static constexpr unsigned FilenameLengthSize = 16U; - static constexpr unsigned LineNumberSize = 16U; + static constexpr unsigned LineNumberSize = 32U; static constexpr unsigned ReferenceTypeSize = 8U; static constexpr unsigned USRLengthSize = 6U; static constexpr unsigned USRBitLengthSize = 8U; Index: clang-tools-extra/clang-doc/BitcodeWriter.h === --- clang-tools-extra/clang-doc/BitcodeWriter.h +++ clang-tools-extra/clang-doc/BitcodeWriter.h @@ -40,7 +40,7 @@ static constexpr unsigned IntSize = 16U; static constexpr unsigned StringLengthSize = 16U; static constexpr unsigned FilenameLengthSize = 16U; - static constexpr unsigned LineNumberSize = 16U; + static constexpr unsigned LineNumberSize = 32U; static constexpr unsigned ReferenceTypeSize = 8U; static constexpr unsigned USRLengthSize = 6U; static constexpr unsigned USRBitLengthSize = 8U; ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits