teemperor added a comment.
I'm mostly putting this up to get some early feedback if anyone sees a problem
with using opaque types here (e.g. it breaks some optimizations, etc.). If it
does, it would still be nice if we could at least make this happen on some
opt-in bases as it would be very beneficial for improving the performance of
LLDB.
================
Comment at: llvm/include/llvm/IR/Instructions.h:1176
->isOpaqueOrPointeeTypeMatches(ResultElementType));
+ assert(PointeeType->isSized());
init(Ptr, IdxList, NameStr);
----------------
)This change and the one below slipped in by accident, that was more of a
debugging help that I wanted to put up as a separate patch.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108407/new/
https://reviews.llvm.org/D108407
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
