[llvm] [GSYM] Callsites: Add data format support and loading from YAML (PR #109781)
via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 27 09:26:30 PDT 2024
================
@@ -214,10 +214,10 @@ TEST(GSYMTest, TestFunctionInfoDecodeErrors) {
FW.writeU32(1); // InfoType::LineTableInfo.
TestFunctionInfoDecodeError(ByteOrder, OutStrm.str(), BaseAddr,
"0x0000000c: missing FunctionInfo InfoType length");
- FW.fixup32(4, FixupOffset); // Write an invalid InfoType enumeration value
+ FW.fixup32(7, FixupOffset); // Write an invalid InfoType enumeration value
FW.writeU32(0); // LineTableInfo InfoType data length.
TestFunctionInfoDecodeError(ByteOrder, OutStrm.str(), BaseAddr,
- "0x00000008: unsupported InfoType 4");
+ "0x00000008: unsupported InfoType 7");
----------------
alx32 wrote:
When this was set to '4', '2' was the highest existent value, so a bit of "headroom" was offered.
I went with the original intent, offering extra headroom for new types without breaking the test.
https://github.com/llvm/llvm-project/pull/109781
More information about the llvm-commits
mailing list