[PATCH] D93659: [AIX][XCOFF] emit vector info of traceback table.
Jason Liu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jun 10 11:26:44 PDT 2021
jasonliu added inline comments.
================
Comment at: llvm/lib/Target/PowerPC/PPCMachineFunctionInfo.cpp:138-141
+ if (Elt == FixedType && !hasVectorParms() && Bits > 31)
+ break;
+ if ((Elt != FixedType || hasVectorParms()) && Bits > 30)
+ break;
----------------
DiggerLin wrote:
> jasonliu wrote:
> > Could we combine the above two if statement?
> can not combine it, for Elt == FixedType && !hasVectorParms() , it only need one bit, so when Bits ==31, we can still put one bit of FixedType which do not have hasVectorParms().
So what would not work for the suggestions I made?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D93659/new/
https://reviews.llvm.org/D93659
More information about the llvm-commits
mailing list