================
@@ -1349,6 +1349,10 @@ static bool upgradeIntrinsicFunction1(Function *F, 
Function *&NewFn,
       else if (Name == "clz.ll" || Name == "popc.ll" || Name == "h2f" ||
                Name == "swap.lo.hi.b64")
         Expand = true;
+      else if (Name == "barrier0" || Name == "barrier.n" ||
----------------
AlexMaclean wrote:

I'm moved these cases as well as the others without a consume_front into a 
StringSwitch in the else branch. I agree we could go further but looks like we 
used to have what you described and switched to this style in 
https://github.com/llvm/llvm-project/commit/b045c36ab92f4ff478dab678aaff4680fbccc7ea
 so maybe there are arguments for both?

https://github.com/llvm/llvm-project/pull/140615
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to