[PATCH] D98030: [IR] Add vscale_range IR function attribute

[Peter Waller via Phabricator via llvm-commits]

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