[PATCH] D75438: [BranchAlign] Reduce the number of emitted fragments due to branch align

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 2 12:55:30 PST 2020


reames requested changes to this revision.
reames added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:454
+
+  // We need to ensure that further data aren't added to the current data
+  // fragment.
----------------
I believe you can simply insert a new empty data fragment instead of adding the special state to all data fragments.  

Also, you only need to do so if the last fragment was a data fragment.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75438





More information about the llvm-commits mailing list