minor cleanup

LuisaGroup · Oct 31, 2024 · 8ef823b · 8ef823b
1 parent 5a0a34e
commit 8ef823b
Show file tree

Hide file tree

Showing 7 changed files with 12 additions and 10 deletions.
diff --git a/include/luisa/xir/ilist.h b/include/luisa/xir/ilist.h
@@ -4,7 +4,6 @@
 #include <luisa/xir/pool.h>
 
 namespace luisa::compute::xir {
-
 template<typename T, typename Base>
 class IntrusiveNode;
 

diff --git a/include/luisa/xir/instructions/branch.h b/include/luisa/xir/instructions/branch.h
@@ -4,12 +4,12 @@
 
 namespace luisa::compute::xir {
 
-class BranchInst : public DerivedBranchInstruction<DerivedInstructionTag::BRANCH> {
+class BranchInst final : public DerivedBranchInstruction<DerivedInstructionTag::BRANCH> {
 public:
     using DerivedBranchInstruction::DerivedBranchInstruction;
 };
 
-class ConditionalBranchInst : public DerivedConditionalBranchInstruction<DerivedInstructionTag::CONDITIONAL_BRANCH> {
+class ConditionalBranchInst final : public DerivedConditionalBranchInstruction<DerivedInstructionTag::CONDITIONAL_BRANCH> {
 public:
     using DerivedConditionalBranchInstruction::DerivedConditionalBranchInstruction;
 };

diff --git a/include/luisa/xir/instructions/outline.h b/include/luisa/xir/instructions/outline.h
@@ -6,8 +6,8 @@ namespace luisa::compute::xir {
 
 class BasicBlock;
 
-class LC_XIR_API OutlineInst final : public DerivedBranchInstruction<DerivedInstructionTag::OUTLINE>,
-                                     public InstructionMergeMixin<OutlineInst> {
+class OutlineInst final : public DerivedBranchInstruction<DerivedInstructionTag::OUTLINE>,
+                          public InstructionMergeMixin<OutlineInst> {
 public:
     using DerivedBranchInstruction::DerivedBranchInstruction;
 };

diff --git a/include/luisa/xir/instructions/unreachable.h b/include/luisa/xir/instructions/unreachable.h
@@ -5,7 +5,7 @@
 namespace luisa::compute::xir {
 
 // Note: this instruction must be the terminator of a basic block.
-class LC_XIR_API UnreachableInst final : public DerivedTerminatorInstruction<DerivedInstructionTag::UNREACHABLE> {
+class UnreachableInst final : public DerivedTerminatorInstruction<DerivedInstructionTag::UNREACHABLE> {
 public:
     using DerivedTerminatorInstruction::DerivedTerminatorInstruction;
 };

diff --git a/include/luisa/xir/metadata.h b/include/luisa/xir/metadata.h
@@ -11,7 +11,7 @@ enum struct DerivedMetadataTag {
     COMMENT,
 };
 
-class LC_XIR_API Metadata : public IntrusiveForwardNode<Metadata> {
+class Metadata : public IntrusiveForwardNode<Metadata> {
 public:
     using Super::Super;
     [[nodiscard]] virtual DerivedMetadataTag derived_metadata_tag() const noexcept = 0;
@@ -49,7 +49,7 @@ template<typename Parent>
 class MetadataMixin {
 
 private:
-    MetadataList _metadata_list;
+    MetadataList _metadata_list{};
 
 private:
     [[nodiscard]] Pool *_get_pool_from_parent() noexcept {

diff --git a/include/luisa/xir/use.h b/include/luisa/xir/use.h
@@ -7,7 +7,7 @@ namespace luisa::compute::xir {
 class Value;
 class User;
 
-class LC_XIR_API Use : public IntrusiveForwardNode<Use> {
+class LC_XIR_API Use final : public IntrusiveForwardNode<Use> {
 
 private:
     User *_user;

diff --git a/src/xir/basic_block.cpp b/src/xir/basic_block.cpp
@@ -3,6 +3,9 @@
 namespace luisa::compute::xir {
 
 BasicBlock::BasicBlock(Pool *pool) noexcept
-    : DerivedValue{pool, nullptr}, _instructions{pool} {}
+    : DerivedValue{pool, nullptr}, _instructions{pool} {
+    _instructions.head_sentinel()->_set_parent_block(this);
+    _instructions.tail_sentinel()->_set_parent_block(this);
+}
 
 }// namespace luisa::compute::xir