[PATCH] D132146: [PowerPC] XCOFF exception section support on the direct assembler path and exception language and reason code lowering from trap intrinsics
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 26 02:41:34 PDT 2022
RKSimon added inline comments.
================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:16247
+ }
+ return Result;
+}
----------------
This is always SDvalue()? If you're never going to create a node and just collect operands, I'd suggest you don't call this 'LowerTargetIntrinsic' and replace it with something more suitable (collectTargetIntrinsicOperands?)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D132146/new/
https://reviews.llvm.org/D132146
More information about the llvm-commits
mailing list