[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 08:24:22 PST 2020


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


================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:446-447
+///   - Otherwise use DS.
+uint8_t X86AsmBackend::choosePrefix(const MCInst &Inst) const {
+  assert((STI.hasFeature(X86::Mode32Bit) || STI.hasFeature(X86::Mode64Bit)) &&
+         "Prefixes can be added only in 32-bit or 64-bit mode.");
----------------
Split the piece of code to D75357


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