[all-commits] [llvm/llvm-project] a79863: Support prefix padding for alignment purposes (Rel...

Philip Reames via All-commits all-commits at lists.llvm.org
Sun Mar 15 19:53:46 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: a79863f2f72747e3abd60326359d0e34f93a6921
      https://github.com/llvm/llvm-project/commit/a79863f2f72747e3abd60326359d0e34f93a6921
  Author: Philip Reames <listmail at philipreames.com>
  Date:   2020-03-15 (Sun, 15 Mar 2020)

  Changed paths:
    M llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp
    A llvm/test/MC/X86/align-via-padding.s
    M llvm/test/MC/X86/align-via-relaxation.s

  Log Message:
  -----------
  Support prefix padding for alignment purposes (Relaxable instructions only)

Now that D75203 has landed and baked for a few days, extend the basic approach to prefix padding as well. The patch itself is fairly straight forward.

For the moment, this patch adds the functional support and some basic testing there of, but defaults to not enabling prefix padding. I want to be able to phrase a separate patch which adds the target specific reasoning and test it cleanly. I haven't decided whether I want to common it with the nop logic or not.

Differential Revision: https://reviews.llvm.org/D75300




More information about the All-commits mailing list