[PATCH] D75268: A light-weight solution to align branches within 32B boundary by prefix padding

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 28 05:53:46 PST 2020


skan marked an inline comment as done.
skan added inline comments.


================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:385
+  // Currently don't deal with Bundle cases.
+  if (OS.getAssembler().isBundlingEnabled())
     return false;
----------------
skan wrote:
> reames wrote:
> > This looks like a potentially unrelated change.  Can it be separated?
> Okay
Separate this piece of code to D75346


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75268





More information about the llvm-commits mailing list