[PATCH] D134195: [PowerPC] XCOFF exception section support on the integrated assembler path

Digger Lin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 21 13:29:19 PDT 2022


DiggerLin added inline comments.


================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:203
+struct ExceptionTableEntry {
+  MCSymbol *Trap;
+  uint64_t TrapAddress;
----------------
I do not think you need to change the content of MCSymbol pointed by *Trap.
```
const MCSymbol *Trap;
```

 will be better.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134195



More information about the llvm-commits mailing list