[PATCH] D108407: [CodeGen][WIP] Avoid generating Record layouts for pointee types

Raphael Isemann via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 19 14:11:21 PDT 2021


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



More information about the cfe-commits mailing list