================ @@ -429,6 +430,15 @@ void InferAddressSpacesImpl::collectRewritableIntrinsicOperands( appendsFlatAddressExpressionToPostorderStack(II->getArgOperand(0), PostorderStack, Visited); break; + case Intrinsic::is_constant: { + Value *Ptr = II->getArgOperand(0); + if (Ptr->getType()->isPtrOrPtrVectorTy()) { + appendsFlatAddressExpressionToPostorderStack(Ptr, PostorderStack, + Visited); + } ---------------- arsenm wrote:
Because unlike a compiler I read by lines, and not statements. The length of that section of the style guide is absurd. It should never be wrong to include braces. That's a lot simpler than parsing out whatever that is https://github.com/llvm/llvm-project/pull/102010 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits