[PATCH] D114075: [IR] Split vscale_range interface

[Bradley Smith via Phabricator via llvm-commits]

bsmith added inline comments.


================
Comment at: llvm/include/llvm/IR/Attributes.h:224
+  /// Returns the optional maximum value for the vscale_range attribute.
+  Optional<unsigned> getVScaleRangeMax() const;
 
----------------
I'm not sure how I feel about this being an Optional type, all of the code changes below are still having to check `max` against `0`, it seems that having this be Optional is just complicating the code for no gain.

Either this should not be Optional, or a value of 0 should be disallowed for max from this interface.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114075/new/

https://reviews.llvm.org/D114075