-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Type] [refactor] Remove LegacyVectorType #1967
[Type] [refactor] Remove LegacyVectorType #1967
Conversation
@@ -36,4 +36,16 @@ Type *TypeFactory::get_pointer_type(Type *element) { | |||
TypeFactory::TypeFactory() { | |||
} | |||
|
|||
DataType TypeFactory::create_vector_or_scalar_type(int width, | |||
DataType element, | |||
bool element_is_pointer) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the core change, we use this function to replace the LegacyVectorType
Codecov Report
@@ Coverage Diff @@
## master #1967 +/- ##
=======================================
Coverage 43.64% 43.64%
=======================================
Files 45 45
Lines 6225 6225
Branches 1106 1106
=======================================
Hits 2717 2717
Misses 3334 3334
Partials 174 174 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM.
Related issue = #1905
In this pr, we remove the class
LegacyVectorType
and use a static functionTypeFactory::create_vector_or_scalar_type
instead.[Click here for the format server]