Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
本 PR 完成宏定义与宏调用落库链路的 schema 对齐和完成了29-32项的宏表实现:
已通过以下方式验证:
macroinvocations.macro_id -> preprocdirects(kind=7) -> preproctext
本次仅对宏定义 + 宏调用链路进行对齐,已把datatable的28. macroinvocations项done,
Update2.27
本次补全 datatable-list 中 29–32 项宏相关表实现:
1.macroparent:
(基于 getImmediateMacroCallerLoc 建立宏调用父子关系)
2.macrolocationbind:
(建立宏调用与 locations 表的绑定关系)
3.macro_argument_unexpanded
4.macro_argument_expanded
已通过以下方式验证:
-表结构生成正常
-count(*) 数据存在
-join 关联正确
-无 orphan 记录
-无重复写入