[PATCH] D150368: TargetExtType: guard against the case that there is no layout type

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 08:10:43 PDT 2023


nikic added a comment.

In D150368#4346325 <https://reviews.llvm.org/D150368#4346325>, @nhaehnle wrote:

> There's nothing right now that would prevent this. Though we could default the layout type to something other than nullptr, if that is preferred. Maybe `void`?

Isn't this already the default? https://github.com/llvm/llvm-project/blob/788e6ff4e5e6414c6707e91e95829f1d7c76b1aa/llvm/lib/IR/Type.cpp#L876


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150368



More information about the llvm-commits mailing list