[PATCH] D69131: [NFC][XCOFF] Using crtp to refactor the xcoff section header

[Jason Liu via Phabricator via llvm-commits]

jasonliu added a comment.

General comments about the direction of this patch:
Previous comments(https://reviews.llvm.org/D68575#inline-617586) suggests to put those two constant value into XCOFF.h. 
I guess the reason for that is we want all the other component of in llvm to be able to use that those two constant value. For example, XCOFFObjectWriter.
But this patch currently put those two values in XCOFFObjectFile.h. The headers in llvm/Object seems to be strictly used only by all the llvm tooling that has an interest in the object itself, but not used by the llvm backend at all.

I think the direction suggested in D68575 <https://reviews.llvm.org/D68575> makes more sense, as those two constant values are more related to object format, and we would want to use them in more than llvm toolings.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D69131