[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
Sun Nov 6 10:16:01 PST 2022
DiggerLin added inline comments.
================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:694
- // when debugging is enabled.
W.write<uint16_t>(SymbolType);
W.write<uint8_t>(StorageClass);
----------------
DiggerLin wrote:
> why delete the comment ?
sorry, I think you maybe have to delete the comment. since you have set "the function indicator" in writeSymbolEntryForCsectMemberLabel.
and I am not sure whether we need to set the "the function indicator" for SymbolType when there is no except auxiliary entry , if there is , we maybe need to add the a TODO comment on it.
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