peterwaller-arm added inline comments.
================ Comment at: llvm/lib/IR/Attributes.cpp:570 + Result += utostr(MinValue); + Result += ','; + Result += utostr(MaxValue); ---------------- Nit: The only other precedent I can see for this is `allocsize`. Grepping the code I found this is always written in tests as `allocsize(x, y)`, however, it prints as `allocsize(x,y)` (no space) when done with `-emit-llvm`, regardless of how it was formatted as input. I figure that is an oversight and this should have the space. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98030/new/ https://reviews.llvm.org/D98030 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits