tyb0807 marked 8 inline comments as done.
tyb0807 added inline comments.
Comment at: llvm/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp:836
+ // inside a bundle to prevent other passes to moving things in between.
+ MIBundleBuilder Bundler(MBB, MBBI);
+ auto &MF = *MBB.getPa
tyb0807 added a comment.
Following Dave's comments above, this patch is now split to 4 different patches
https://reviews.llvm.org/D117753
https://reviews.llvm.org/D117757
https://reviews.llvm.org/D117763
https://reviews.llvm.org/D117764
and may now be abandoned.
Repository:
rG LLVM Github Mon
dmgreen added a comment.
This looks like a large patch. It may be better split up into a base/isel part,
a clang part and a global isel part (which may require different reviewers).
Comment at: llvm/lib/Target/AArch64/AArch64ExpandPseudoInsts.cpp:836
+ // inside a bundle to p