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

WÁNG Xuěruì via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 17 01:08:01 PDT 2022


xen0n marked an inline comment as done.
xen0n added inline comments.


================
Comment at: llvm/lib/Target/LoongArch/LoongArchISelLowering.cpp:181
+  MachineFunction &MF = DAG.getMachineFunction();
+  bool Size = Subtarget.getGRLen() / 8;
+  int FI = MF.getFrameInfo().CreateFixedObject(Size, 0, false);
----------------
wangleiat wrote:
> Incorrect type
Oh my. I was pretty sure I used `auto` before, not sure how I managed to keep the wrong "bool" type (of IsRISCV64) in the submission... Fixed though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132012



More information about the llvm-commits mailing list