[Mlir-commits] [mlir] [mlir] Make single value `ValueRange`s memory safer (PR #121996)

[Jeff Niu]

Markus =?utf-8?q?Böck?= <markus.boeck02 at gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/121996 at github.com>


https://github.com/Mogball commented:

I am conflicted about this. On one hand, it seems like it's not a bad thing to have. On the other hand, I already think about `ValueRange` and friends like `ArrayRef<Value>`, and it's not valid to do 

```C++
ArrayRef<Value> stuff = getTemporary();
```

https://github.com/llvm/llvm-project/pull/121996