Rebased on the NFC patch;

In D87029#2285398, @jasonliu wrote:

I think it would help the review if we could put the NFC portion(e.g. TypeSize -> StorageUnitSize) to a new patch, and give some rationale about the NFC change.

Updated the comments;

Addressed the comments;

Addressed the comments;

Do we want to take advantage of this patch to move those checks into the cmake configure so we don't need to duplicate the logic in 2 files as @sfertile mentioned previously?

Rebased on the pragma/pack patch;
Added packed related testcase for bitfield;

Fix the PragmaStack is empty assertion failure when we do: #pragma pack(2) then #pragma align(reset)

Use default argument;

Removed redundant header file include;

In D86790#2255371, @jyknight wrote:

Do you have open questions on whether some callsites passing "false" here, should be switched to true? Given what's here, I would say that it definitely does not makes sense to add this parameter everywhere.

ping.

Add a format indicator as a part of function names;

Add a separator between PID and timestamp;

Use itostr for signed integer value ;

Addressed the comments;
Fixed the testcase;

Rebase on the latest master;

LGTM. thanks.

so that code compiled with large code model TOC reference would not
take up spaces from the normal small code model TOC reference.

As you mentioned in the summary, this patch seems be addressing two orthogonal things:

1. remove SC from MCSectionXCOFF and let it only be a property of MCSymbolXCOFF
2. when branching to an undefined function, branch to qualname instead.

Added descriptions for struct and functions;
Addressed other comments;

LGTM. thanks

Refactor the code by making emitXXStructorList a virtual function;
Added one testcase for .ref;
Addressed other comments;

Removed the disablement in IncrementalProcessingTest.cpp cross-target test;

Created alias for sinit and sterm;
Adjusted the testcase accordingly;

LGTM. Thanks.

LGTM. thanks.

Fix clang-tidy errors;

• Simplified the test command line;
• Split the typedef related tests into two to address the LIT testcase failure on windows platform;

Add one more testcase;
Addressed other comments;

LGTM. Thanks

LGTM with minor nit.

Adjust the query;

Simplified the testcase;

Removed unused var;

Set Handled... = true for non-AIX power alignment;
Addressed other comments;

Fixed a base class related case by adding IsFirstNonEmpty flag;
Split the aix-Wpacked.cpp testcase into two;
Addressed other comments;

Fixed a -Wpacked related case and added the case to the tests;
Fixed the base class related code issue;
Addressed other comments;