[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
Thu Feb 27 20:14:50 PST 2020
skan marked 2 inline comments 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;
----------------
reames wrote:
> This looks like a potentially unrelated change. Can it be separated?
Okay
================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:493
+ case X86::CS:
+ return 0x2e;
+ case X86::SS:
----------------
reames wrote:
> Please replace constants with X86::CS_PREFIX and friends.
X86::CS_PREFIX is a enum value and used to represent the instruction is a cs prefix indeed. Constants 0x2e is the encoding value for X86::CS, thery are different.
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