[PATCH] D157565: [CodeGen] Add AArch64 behavior to existing MFS tests
Rahman Lavaee via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 17 16:31:32 PDT 2023
rahmanl added inline comments.
================
Comment at: llvm/lib/CodeGen/BasicBlockSections.cpp:294
+ continue;
+ NopInstr.addOperand(
+ MachineOperand::CreateImm(Nop.getOperand(0).getImm()));
----------------
dhoekwater wrote:
> rahmanl wrote:
> > rahmanl wrote:
> > > Is this because of ARM? Maybe add a comment to explain the rationale.
> > I see, this is target specific now. Although this code was target-specific before too, but we only wanted code for X86. Do you think we should implement insertNoop (https://llvm.org/doxygen/classllvm_1_1TargetInstrInfo.html#a4ee57d5d6295dfeb44f3b55301b20020) for X86 and AArch64 now?
> Yes! That's exactly what I was looking for, but I couldn't find the function since I was looking for something like "addNop". Should we do that in this patch or should I make another?
Right. Separate the entire `avoidZeroOffsetLandingPad` changes into its own patch if you can. You might get new reviewers when implement `TargetInstrInfo`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D157565/new/
https://reviews.llvm.org/D157565
More information about the llvm-commits
mailing list