[llvm-commits] [PATCH] MCFragments Clean Up

David Meyer pdox at google.com
Thu Oct 14 11:48:29 PDT 2010


Hi Dan,

This patch against rev 116493 does the following:

* Adds four virtual methods to MCFragment:
    ComputeSize() - Compute the size of this fragment
    WriteData() - Write the data of this fragment
    AssertLegalInVirtualSection() - Assert that this fragment is legal
in a virtual section
    dump_kind_data() - Dump the kind-specific data for this fragment

* Moves the implementation of these actions from MCAssembler.cpp to a
new file "MCFragment.cpp"

* As a benefit, most of the switches on MCFragment::Kind and many of
the MCFragment casts go away.

* Introduces the PendingLayout flag to make it possible to reuse the MCAsmLayout
getter functions (i.e. getFragmentAddress() and getFragmentOffset())
while inside of
ComputeSize().

Thanks. Please let me know what you think.

- David M
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mcfragment-clean-up.patch
Type: application/octet-stream
Size: 29796 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101014/445e1433/attachment.obj>


More information about the llvm-commits mailing list