Thank you @manupa-arm for this input! I'll go over the points in more detail in the next few days.
For A1, I agree, that enums are generally a cleaner solution. We stuck with ints until now because of the the following two reasons: - Phases for TIR passes are currently also int-based - For some Phases we struggled to find concise names I think, if we are fine with generic names for some phases until we find good names, we can switch to enums. For A2, I also prefer A2.1. I was prototyping something along the line and will get back with more details (and an RFC update) once it's done. -- Reply to this email directly or view it on GitHub: https://github.com/apache/tvm-rfcs/pull/60#issuecomment-1124022013 You are receiving this because you are subscribed to this thread. Message ID: <apache/tvm-rfcs/pull/60/c1124022...@github.com>
