[PATCH] D132012: [LoongArch] Add support for llvm.eh.dwarf.cfa

[Lu Weining via Phabricator via llvm-commits]

SixWeining accepted this revision.
SixWeining added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:181
+  MachineFunction &MF = DAG.getMachineFunction();
+  auto Size = Subtarget.getGRLen() / 8;
+  auto FI = MF.getFrameInfo().CreateFixedObject(Size, 0, false);
----------------
Seems this is only used one time. Better               
to inline it? But I can’t find this rule in llvm coding standard.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132012