HomePhabricator

[X86][MC] Make -x86-pad-max-prefix-size compatible with --mc-relax-all
Audit Required5d73f79c5478

Authored by skan on Apr 10 2020, 3:37 AM.

Description

[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

Details

Auditors
grosbach
Committed
skanApr 10 2020, 8:30 PM
Reviewer
MaskRay
Differential Revision
D77851: [X86][MC] Make -x86-pad-max-prefix-size compatible with --mc-relax-all
Parents
rGc610807afe50: [lit] Temporarily disable failing tests on Windows
Branches
Unknown
Tags
Unknown