[Openmp-commits] [PATCH] D103090: [libomptarget][nfc][amdgpu] Factor out setting upper bounds

[Dhruva Chakrabarti via Phabricator via Openmp-commits]

dhruvachak added inline comments.


================
Comment at: openmp/libomptarget/plugins/amdgpu/src/rtl.cpp:716
+    return true;
+  } else {
+    return false;
----------------
How about making the change that clang-tidy is suggesting here?

Another question: Any specific reason why you are not using std::min? Is it because emitting the DP becomes a bit cumbersome?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103090