[PATCH] D108961: [RISCV] MC relaxation for out-of-range conditional branch.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 2 10:42:15 PDT 2021


MaskRay added a comment.

In D108961#2980290 <https://reviews.llvm.org/D108961#2980290>, @craig.topper wrote:

> Will the BranchRelaxation pass account for .insn directives in inline asm correctly? Or will we need to do additional work for that associated with D108602 <https://reviews.llvm.org/D108602>

I haven't looked at it closely yet. BranchRelaxation.cpp uses `TargetInstrInfo::getInstSizeInBytes` to compute the size.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108961



More information about the llvm-commits mailing list