[PATCH] D75357: [X86] Add a private member function determinePaddingPrefix for X86AsmBackend

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 2 22:25:42 PST 2020


skan added a comment.

In D75357#1902293 <https://reviews.llvm.org/D75357#1902293>, @MaskRay wrote:

> Do you have a plan how `determinePaddingPrefix` will be used? Without a plan, there may be some churn that moves things around. The plan should also include how the complex logic will be tested eventually.


Philip prototyped what prefix padding for relaxable instructions might look like in D75203 <https://reviews.llvm.org/D75203>, `determinePaddingPrefix` is planed to be used to determine the appropriate prefix in the framework.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75357





More information about the llvm-commits mailing list