[clang] [mlir] [mlir][acc] Use consistent name for device_num operand (PR #136745)

[Razvan Lupusoru via cfe-commits]

razvanlupusoru wrote:

> Also, we have `getIfCondMutable` in a number of places as well, which is now inconsistent with skipping the 'operand'.

I am not sure I follow - are you saying that for consistency, you think that the word 'operand' should be included in the name of `ifCond` (or excluded for other cases)? We do have multiple cases where we do not use the word 'operand' - for example in acc.parallel we have numGangs, numWorkers, vectorLength, selfCond, ifCond. It is used in cases where it looks awkward to not have it. For example `asyncOperands` and `waitOperands`. This is subjective though :)

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