[llvm] [InlineAsm] wrap ConstraintCode in enum class NFC (PR #66003)

Jessica Clarke via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 11 23:16:01 PDT 2023


================
@@ -2521,13 +2521,13 @@ AVRTargetLowering::getConstraintType(StringRef Constraint) const {
   return TargetLowering::getConstraintType(Constraint);
 }
 
-unsigned
+InlineAsm::ConstraintCode
 AVRTargetLowering::getInlineAsmMemConstraint(StringRef ConstraintCode) const {
   // Not sure if this is actually the right thing to do, but we got to do
   // *something* [agnat]
   switch (ConstraintCode[0]) {
----------------
jrtc27 wrote:

Disagree. This causes much less churn if and when new cases need to be added.

https://github.com/llvm/llvm-project/pull/66003


More information about the llvm-commits mailing list