Commit c5f10123 authored by Seth Brenith's avatar Seth Brenith Committed by Commit Bot

[torque] Don't generate field lists for classes with undefined layout

Minor cleanup: some classes in Torque don't yet have any definitions for
their fields, so it doesn't make sense to emit field layout macros for
those classes.

Bug: v8:7793
Change-Id: Iee38aa3cbe684f4a63329a676e2e94944dc05de1
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/1925010Reviewed-by: 's avatarTobias Tebbi <tebbi@chromium.org>
Commit-Queue: Seth Brenith <seth.brenith@microsoft.com>
Cr-Commit-Position: refs/heads/master@{#65054}
parent f2a74165
...@@ -3037,7 +3037,7 @@ void ImplementationVisitor::GenerateClassFieldOffsets( ...@@ -3037,7 +3037,7 @@ void ImplementationVisitor::GenerateClassFieldOffsets(
// TODO(danno): Remove this once all classes use ClassFieldOffsetGenerator // TODO(danno): Remove this once all classes use ClassFieldOffsetGenerator
// to generate field offsets without the use of macros. // to generate field offsets without the use of macros.
if (!type->GenerateCppClassDefinitions()) { if (!type->GenerateCppClassDefinitions() && !type->HasUndefinedLayout()) {
MacroFieldOffsetsGenerator g(header, type); MacroFieldOffsetsGenerator g(header, type);
for (auto f : type->fields()) { for (auto f : type->fields()) {
CurrentSourcePosition::Scope scope(f.pos); CurrentSourcePosition::Scope scope(f.pos);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment