efriedma added a comment.

I don't want to change with the distinction between FunctionProtoType and 
FunctionNoProtoType; that would involve auditing a bunch of code we don't 
really want to mess with.

FunctionProtoType and FunctionNoProtoType don't have any member variables at 
the moment, though.  The only difference is that FunctionProtoType has 
TrailingObjects, and FunctionNoProtoType doesn't. It should be fine to just 
require the same TrailingObjects for FunctionNoProtoType.  So FunctionProtoType 
and FunctionNoProtoType would have the same memory layout.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124998

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to