Skip to content

Commit 9544206

Browse files
author
Amirhossein Pashaeehir
committed
Remove UnwindTable dependency on CIE, and FDE (transitively on llvm/Object)
For creating new UnwindTable, two static methods was implemented inside it, to create an instance of it from a CIE or FDE. This static methods are moved out of the class as a library functions.
1 parent cc68367 commit 9544206

File tree

3 files changed

+71
-64
lines changed

3 files changed

+71
-64
lines changed

‎llvm/include/llvm/DebugInfo/DWARF/DWARFDebugFrame.h

Lines changed: 36 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -310,9 +310,6 @@ class UnwindRow {
310310

311311
raw_ostream &operator<<(raw_ostream &OS, const UnwindRow &Row);
312312

313-
class CIE;
314-
class FDE;
315-
316313
/// A class that contains all UnwindRow objects for an FDE or a single unwind
317314
/// row for a CIE. To unwind an address the rows, which are sorted by start
318315
/// address, can be searched to find the UnwindRow with the lowest starting
@@ -333,6 +330,12 @@ class UnwindTable {
333330
assert(Index < size());
334331
return Rows[Index];
335332
}
333+
void insertRow(const UnwindRow &Row) { Rows.push_back(Row); }
334+
335+
/// Set the last address that this unwinding table refers to.
336+
///
337+
/// This is used when this table is created based on a FDE.
338+
void setEndAddress(uint64_t Addr) { EndAddress = Addr; }
336339

337340
/// Dump the UnwindTable to the stream.
338341
///
@@ -351,32 +354,6 @@ class UnwindTable {
351354
void dump(raw_ostream &OS, DIDumpOptions DumpOpts,
352355
unsigned IndentLevel = 0) const;
353356

354-
/// Create an UnwindTable from a Common Information Entry (CIE).
355-
///
356-
/// \param Cie The Common Information Entry to extract the table from. The
357-
/// CFIProgram is retrieved from the \a Cie object and used to create the
358-
/// UnwindTable.
359-
///
360-
/// \returns An error if the DWARF Call Frame Information opcodes have state
361-
/// machine errors, or a valid UnwindTable otherwise.
362-
static Expected<UnwindTable> create(const CIE *Cie);
363-
364-
/// Create an UnwindTable from a Frame Descriptor Entry (FDE).
365-
///
366-
/// \param Fde The Frame Descriptor Entry to extract the table from. The
367-
/// CFIProgram is retrieved from the \a Fde object and used to create the
368-
/// UnwindTable.
369-
///
370-
/// \returns An error if the DWARF Call Frame Information opcodes have state
371-
/// machine errors, or a valid UnwindTable otherwise.
372-
static Expected<UnwindTable> create(const FDE *Fde);
373-
374-
private:
375-
RowContainer Rows;
376-
/// The end address when data is extracted from a FDE. This value will be
377-
/// invalid when a UnwindTable is extracted from a CIE.
378-
std::optional<uint64_t> EndAddress;
379-
380357
/// Parse the information in the CFIProgram and update the CurrRow object
381358
/// that the state machine describes.
382359
///
@@ -394,10 +371,40 @@ class UnwindTable {
394371
/// DW_CFA_restore and DW_CFA_restore_extended opcodes.
395372
Error parseRows(const CFIProgram &CFIP, UnwindRow &CurrRow,
396373
const RegisterLocations *InitialLocs);
374+
375+
private:
376+
RowContainer Rows;
377+
/// The end address when data is extracted from a FDE. This value will be
378+
/// invalid when a UnwindTable is extracted from a CIE.
379+
std::optional<uint64_t> EndAddress;
397380
};
398381

399382
raw_ostream &operator<<(raw_ostream &OS, const UnwindTable &Rows);
400383

384+
class CIE;
385+
386+
/// Create an UnwindTable from a Common Information Entry (CIE).
387+
///
388+
/// \param Cie The Common Information Entry to extract the table from. The
389+
/// CFIProgram is retrieved from the \a Cie object and used to create the
390+
/// UnwindTable.
391+
///
392+
/// \returns An error if the DWARF Call Frame Information opcodes have state
393+
/// machine errors, or a valid UnwindTable otherwise.
394+
Expected<UnwindTable> createUnwindTable(const CIE *Cie);
395+
396+
class FDE;
397+
398+
/// Create an UnwindTable from a Frame Descriptor Entry (FDE).
399+
///
400+
/// \param Fde The Frame Descriptor Entry to extract the table from. The
401+
/// CFIProgram is retrieved from the \a Fde object and used to create the
402+
/// UnwindTable.
403+
///
404+
/// \returns An error if the DWARF Call Frame Information opcodes have state
405+
/// machine errors, or a valid UnwindTable otherwise.
406+
Expected<UnwindTable> createUnwindTable(const FDE *Fde);
407+
401408
/// An entry in either debug_frame or eh_frame. This entry can be a CIE or an
402409
/// FDE.
403410
class FrameEntry {

‎llvm/lib/DebugInfo/DWARF/DWARFDebugFrame.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ raw_ostream &llvm::dwarf::operator<<(raw_ostream &OS, const UnwindTable &Rows) {
201201
return OS;
202202
}
203203

204-
Expected<UnwindTable> UnwindTable::create(const FDE *Fde) {
204+
Expected<UnwindTable> llvm::dwarf::createUnwindTable(const FDE *Fde) {
205205
const CIE *Cie = Fde->getLinkedCIE();
206206
if (Cie == nullptr)
207207
return createStringError(errc::invalid_argument,
@@ -215,7 +215,7 @@ Expected<UnwindTable> UnwindTable::create(const FDE *Fde) {
215215
UnwindTable UT;
216216
UnwindRow Row;
217217
Row.setAddress(Fde->getInitialLocation());
218-
UT.EndAddress = Fde->getInitialLocation() + Fde->getAddressRange();
218+
UT.setEndAddress(Fde->getInitialLocation() + Fde->getAddressRange());
219219
if (Error CieError = UT.parseRows(Cie->cfis(), Row, nullptr))
220220
return std::move(CieError);
221221
// We need to save the initial locations of registers from the CIE parsing
@@ -227,11 +227,11 @@ Expected<UnwindTable> UnwindTable::create(const FDE *Fde) {
227227
// Do not add that to the unwind table.
228228
if (Row.getRegisterLocations().hasLocations() ||
229229
Row.getCFAValue().getLocation() != UnwindLocation::Unspecified)
230-
UT.Rows.push_back(Row);
230+
UT.insertRow(Row);
231231
return UT;
232232
}
233233

234-
Expected<UnwindTable> UnwindTable::create(const CIE *Cie) {
234+
Expected<UnwindTable> llvm::dwarf::createUnwindTable(const CIE *Cie) {
235235
// Rows will be empty if there are no CFI instructions.
236236
if (Cie->cfis().empty())
237237
return UnwindTable();
@@ -244,7 +244,7 @@ Expected<UnwindTable> UnwindTable::create(const CIE *Cie) {
244244
// Do not add that to the unwind table.
245245
if (Row.getRegisterLocations().hasLocations() ||
246246
Row.getCFAValue().getLocation() != UnwindLocation::Unspecified)
247-
UT.Rows.push_back(Row);
247+
UT.insertRow(Row);
248248
return UT;
249249
}
250250

@@ -603,7 +603,7 @@ void CIE::dump(raw_ostream &OS, DIDumpOptions DumpOpts) const {
603603
CFIs.dump(OS, DumpOpts, /*IndentLevel=*/1, /*InitialLocation=*/{});
604604
OS << "\n";
605605

606-
if (Expected<UnwindTable> RowsOrErr = UnwindTable::create(this))
606+
if (Expected<UnwindTable> RowsOrErr = createUnwindTable(this))
607607
RowsOrErr->dump(OS, DumpOpts, 1);
608608
else {
609609
DumpOpts.RecoverableErrorHandler(joinErrors(
@@ -631,7 +631,7 @@ void FDE::dump(raw_ostream &OS, DIDumpOptions DumpOpts) const {
631631
CFIs.dump(OS, DumpOpts, /*IndentLevel=*/1, InitialLocation);
632632
OS << "\n";
633633

634-
if (Expected<UnwindTable> RowsOrErr = UnwindTable::create(this))
634+
if (Expected<UnwindTable> RowsOrErr = createUnwindTable(this))
635635
RowsOrErr->dump(OS, DumpOpts, 1);
636636
else {
637637
DumpOpts.RecoverableErrorHandler(joinErrors(

‎llvm/unittests/DebugInfo/DWARF/DWARFDebugFrameTest.cpp

Lines changed: 28 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -465,9 +465,9 @@ TEST(DWARFDebugFrame, UnwindTableEmptyRows) {
465465
EXPECT_THAT_ERROR(parseCFI(TestCIE, {}), Succeeded());
466466
EXPECT_TRUE(TestCIE.cfis().empty());
467467

468-
// Verify dwarf::UnwindTable::create() won't result in errors and
468+
// Verify dwarf::createUnwindTable() won't result in errors and
469469
// and empty rows are not added to CIE UnwindTable.
470-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestCIE);
470+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestCIE);
471471
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
472472
const size_t ExpectedNumOfRows = 0;
473473
EXPECT_EQ(RowsOrErr->size(), ExpectedNumOfRows);
@@ -486,9 +486,9 @@ TEST(DWARFDebugFrame, UnwindTableEmptyRows) {
486486
EXPECT_THAT_ERROR(parseCFI(TestFDE, {}), Succeeded());
487487
EXPECT_TRUE(TestFDE.cfis().empty());
488488

489-
// Verify dwarf::UnwindTable::create() won't result in errors and
489+
// Verify dwarf::createUnwindTable() won't result in errors and
490490
// and empty rows are not added to FDE UnwindTable.
491-
RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
491+
RowsOrErr = dwarf::createUnwindTable(&TestFDE);
492492
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
493493
EXPECT_EQ(RowsOrErr->size(), ExpectedNumOfRows);
494494
}
@@ -504,9 +504,9 @@ TEST(DWARFDebugFrame, UnwindTableEmptyRows_NOPs) {
504504
EXPECT_THAT_ERROR(parseCFI(TestCIE, {dwarf::DW_CFA_nop}), Succeeded());
505505
EXPECT_TRUE(!TestCIE.cfis().empty());
506506

507-
// Verify dwarf::UnwindTable::create() won't result in errors and
507+
// Verify dwarf::createUnwindTable() won't result in errors and
508508
// and empty rows are not added to CIE UnwindTable.
509-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestCIE);
509+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestCIE);
510510
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
511511
const size_t ExpectedNumOfRows = 0;
512512
EXPECT_EQ(RowsOrErr->size(), ExpectedNumOfRows);
@@ -525,9 +525,9 @@ TEST(DWARFDebugFrame, UnwindTableEmptyRows_NOPs) {
525525
EXPECT_THAT_ERROR(parseCFI(TestFDE, {dwarf::DW_CFA_nop}), Succeeded());
526526
EXPECT_TRUE(!TestFDE.cfis().empty());
527527

528-
// Verify dwarf::UnwindTable::create() won't result in errors and
528+
// Verify dwarf::createUnwindTable() won't result in errors and
529529
// and empty rows are not added to FDE UnwindTable.
530-
RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
530+
RowsOrErr = dwarf::createUnwindTable(&TestFDE);
531531
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
532532
EXPECT_EQ(RowsOrErr->size(), ExpectedNumOfRows);
533533
}
@@ -567,7 +567,7 @@ TEST(DWARFDebugFrame, UnwindTableErrorNonAscendingFDERows) {
567567
Succeeded());
568568

569569
// Verify we catch state machine error.
570-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
570+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
571571
EXPECT_THAT_ERROR(RowsOrErr.takeError(),
572572
FailedWithMessage("DW_CFA_set_loc with adrress 0x1000 which"
573573
" must be greater than the current row "
@@ -603,7 +603,7 @@ TEST(DWARFDebugFrame, UnwindTableError_DW_CFA_restore_state) {
603603
Succeeded());
604604

605605
// Verify we catch state machine error.
606-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
606+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
607607
EXPECT_THAT_ERROR(RowsOrErr.takeError(),
608608
FailedWithMessage("DW_CFA_restore_state without a matching "
609609
"previous DW_CFA_remember_state"));
@@ -639,7 +639,7 @@ TEST(DWARFDebugFrame, UnwindTableError_DW_CFA_GNU_window_save) {
639639
Succeeded());
640640

641641
// Verify we catch state machine error.
642-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
642+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
643643
EXPECT_THAT_ERROR(RowsOrErr.takeError(),
644644
FailedWithMessage("DW_CFA opcode 0x2d is not supported for "
645645
"architecture x86_64"));
@@ -674,7 +674,7 @@ TEST(DWARFDebugFrame, UnwindTableError_DW_CFA_def_cfa_offset) {
674674
Succeeded());
675675

676676
// Verify we catch state machine error.
677-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
677+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
678678
EXPECT_THAT_ERROR(RowsOrErr.takeError(),
679679
FailedWithMessage("DW_CFA_def_cfa_offset found when CFA "
680680
"rule was not RegPlusOffset"));
@@ -709,7 +709,7 @@ TEST(DWARFDebugFrame, UnwindTableDefCFAOffsetSFCFAError) {
709709
Succeeded());
710710

711711
// Verify we catch state machine error.
712-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
712+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
713713
EXPECT_THAT_ERROR(RowsOrErr.takeError(),
714714
FailedWithMessage("DW_CFA_def_cfa_offset_sf found when CFA "
715715
"rule was not RegPlusOffset"));
@@ -745,7 +745,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_def_cfa_register) {
745745
EXPECT_THAT_ERROR(parseCFI(TestFDE, {}), Succeeded());
746746

747747
// Verify we catch state machine error.
748-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
748+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
749749
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
750750
const dwarf::UnwindTable &Rows = RowsOrErr.get();
751751
EXPECT_EQ(Rows.size(), 1u);
@@ -817,7 +817,7 @@ TEST(DWARFDebugFrame, UnwindTableRowPushingOpcodes) {
817817
Reg, dwarf::UnwindLocation::createIsRegisterPlusOffset(InReg, 0));
818818

819819
// Verify we catch state machine error.
820-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
820+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
821821
ASSERT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
822822
const dwarf::UnwindTable &Rows = RowsOrErr.get();
823823
EXPECT_EQ(Rows.size(), 6u);
@@ -892,7 +892,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_restore) {
892892
Reg, dwarf::UnwindLocation::createIsRegisterPlusOffset(InReg, 0));
893893

894894
// Verify we catch state machine error.
895-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
895+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
896896
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
897897
const dwarf::UnwindTable &Rows = RowsOrErr.get();
898898
EXPECT_EQ(Rows.size(), 2u);
@@ -955,7 +955,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_restore_extended) {
955955
Reg, dwarf::UnwindLocation::createIsRegisterPlusOffset(InReg, 0));
956956

957957
// Verify we catch state machine error.
958-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
958+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
959959
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
960960
const dwarf::UnwindTable &Rows = RowsOrErr.get();
961961
EXPECT_EQ(Rows.size(), 2u);
@@ -1016,7 +1016,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_offset) {
10161016
Reg3, dwarf::UnwindLocation::createAtCFAPlusOffset(8));
10171017

10181018
// Verify we catch state machine error.
1019-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1019+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
10201020
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
10211021
const dwarf::UnwindTable &Rows = RowsOrErr.get();
10221022
EXPECT_EQ(Rows.size(), 1u);
@@ -1068,7 +1068,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_val_offset) {
10681068
Reg2, dwarf::UnwindLocation::createIsCFAPlusOffset(8));
10691069

10701070
// Verify we catch state machine error.
1071-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1071+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
10721072
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
10731073
const dwarf::UnwindTable &Rows = RowsOrErr.get();
10741074
EXPECT_EQ(Rows.size(), 1u);
@@ -1113,7 +1113,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_nop) {
11131113
Reg1, dwarf::UnwindLocation::createAtCFAPlusOffset(-8));
11141114

11151115
// Verify we catch state machine error.
1116-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1116+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
11171117
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
11181118
const dwarf::UnwindTable &Rows = RowsOrErr.get();
11191119
EXPECT_EQ(Rows.size(), 1u);
@@ -1203,7 +1203,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_remember_state) {
12031203
Reg3, dwarf::UnwindLocation::createAtCFAPlusOffset(-24));
12041204

12051205
// Verify we catch state machine error.
1206-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1206+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
12071207
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
12081208
const dwarf::UnwindTable &Rows = RowsOrErr.get();
12091209
EXPECT_EQ(Rows.size(), 5u);
@@ -1270,7 +1270,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_undefined) {
12701270
dwarf::UnwindLocation::createUndefined());
12711271

12721272
// Verify we catch state machine error.
1273-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1273+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
12741274
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
12751275
const dwarf::UnwindTable &Rows = RowsOrErr.get();
12761276
EXPECT_EQ(Rows.size(), 1u);
@@ -1314,7 +1314,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_same_value) {
13141314
VerifyLocs.setRegisterLocation(Reg1, dwarf::UnwindLocation::createSame());
13151315

13161316
// Verify we catch state machine error.
1317-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1317+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
13181318
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
13191319
const dwarf::UnwindTable &Rows = RowsOrErr.get();
13201320
EXPECT_EQ(Rows.size(), 1u);
@@ -1360,7 +1360,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_register) {
13601360
Reg, dwarf::UnwindLocation::createIsRegisterPlusOffset(InReg, 0));
13611361

13621362
// Verify we catch state machine error.
1363-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1363+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
13641364
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
13651365
const dwarf::UnwindTable &Rows = RowsOrErr.get();
13661366
EXPECT_EQ(Rows.size(), 1u);
@@ -1412,7 +1412,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_expression) {
14121412
Reg, dwarf::UnwindLocation::createAtDWARFExpression(Expr));
14131413

14141414
// Verify we catch state machine error.
1415-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1415+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
14161416
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
14171417
const dwarf::UnwindTable &Rows = RowsOrErr.get();
14181418
EXPECT_EQ(Rows.size(), 1u);
@@ -1464,7 +1464,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_val_expression) {
14641464
Reg, dwarf::UnwindLocation::createIsDWARFExpression(Expr));
14651465

14661466
// Verify we catch state machine error.
1467-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1467+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
14681468
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
14691469
const dwarf::UnwindTable &Rows = RowsOrErr.get();
14701470
EXPECT_EQ(Rows.size(), 1u);
@@ -1527,7 +1527,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_def_cfa) {
15271527
Reg, dwarf::UnwindLocation::createIsRegisterPlusOffset(InReg, 0));
15281528

15291529
// Verify we catch state machine error.
1530-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1530+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
15311531
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
15321532
const dwarf::UnwindTable &Rows = RowsOrErr.get();
15331533
EXPECT_EQ(Rows.size(), 5u);
@@ -1625,7 +1625,7 @@ TEST(DWARFDebugFrame, UnwindTable_DW_CFA_LLVM_def_aspace_cfa) {
16251625
Reg, dwarf::UnwindLocation::createIsRegisterPlusOffset(InReg, 0));
16261626

16271627
// Verify we catch state machine error.
1628-
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::UnwindTable::create(&TestFDE);
1628+
Expected<dwarf::UnwindTable> RowsOrErr = dwarf::createUnwindTable(&TestFDE);
16291629
EXPECT_THAT_ERROR(RowsOrErr.takeError(), Succeeded());
16301630
const dwarf::UnwindTable &Rows = RowsOrErr.get();
16311631
EXPECT_EQ(Rows.size(), 5u);

0 commit comments

Comments
 (0)