[PATCH] D80249: WIP: CodeGen: Don't lazily construct MachineFunctionInfo

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D80249#2107516 <https://reviews.llvm.org/D80249#2107516>, @kparzysz wrote:

> A "create" function shouldn't return a null pointer.  I think there should be a default MFI that has nothing in it, and it would be what the `createMachineFunctionInfo` returns a pointer to in the absence of overrides.  It could even be statically allocated since is has no members:
>
>   virtual MachineFunctionInfo *
>   createMachineFunctionInfo(BumpPtrAllocator &Allocator, const Function &F,
>                             const TargetSubtargetInfo *STI) const {
>     static MachineFunctionInfo default;
>     return &default;
>   }
>
>
> Edit: fix indentation in the code


Alternatively, make it pure virtual or make it llvm_unreachable?


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

https://reviews.llvm.org/D80249