[all-commits] [llvm/llvm-project] 5d73f7: [X86][MC] Make -x86-pad-max-prefix-size compatible...

KanRobert via All-commits all-commits at lists.llvm.org
Fri Apr 10 20:30:44 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 5d73f79c54788bc07dabe9ed75c36f12bafe9d28
      https://github.com/llvm/llvm-project/commit/5d73f79c54788bc07dabe9ed75c36f12bafe9d28
  Author: Shengchen Kan <shengchen.kan at intel.com>
  Date:   2020-04-11 (Sat, 11 Apr 2020)

  Changed paths:
    M llvm/lib/MC/MCObjectStreamer.cpp
    M llvm/test/MC/X86/align-branch-64-relax-all.s

  Log Message:
  -----------
  [X86][MC] Make -x86-pad-max-prefix-size compatible with --mc-relax-all

Summary: We allow non-relaxable instructions emitted into relaxable Fragment when we prefix padding branch. So we need to check if the instruction need relaxation before relaxing it.  Without this patch, it currently triggers a `report_fatal_error` in `llvm::MCAsmBackend::relaxInstruction` when we prefix padding branch along with `--mc-relax-all`.

Reviewers: LuoYuanke, reames, MaskRay

Reviewed By: MaskRay

Subscribers: MaskRay, hiraditya, llvm-commits

Tags: #llvm

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




More information about the All-commits mailing list