================ @@ -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