[PATCH] D120861: [XCOFF][1/3] support fileHeader and sectionHeaders in 64-bit XCOFF.
Digger Lin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Mar 18 08:44:02 PDT 2022
DiggerLin accepted this revision.
DiggerLin added a comment.
This revision is now accepted and ready to land.
LGTM.
I do not have further comment, but please wait for James approve.
================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:299
bool
needsAuxiliaryHeader() const { /* TODO aux header support not implemented. */
return false;
----------------
jhenderson wrote:
> Put the TODO comment either on a new line within the function (as suggested inline), or immediately before it as a commetnt, using `//` rather than block comments, then use clang-format to reformat it appropriately.
James ask for put the comment within the function , not sure whether meet James comment. @jhenderson
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120861/new/
https://reviews.llvm.org/D120861
More information about the llvm-commits
mailing list