[PATCH] D137138: AMDGPU: Fix DivergenceAnalysis for llvm.read_register

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 1 12:23:15 PDT 2022


rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp:876
+
+  // Go through the inline asm API since I don't see a better way.
+  std::string ConstraintStr = "{" + RegName.str() + "}";
----------------
You could just check the first symbol of the name, it is divergent if it is 'v' or 'a'. Same effect at the end, but much less work.


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

https://reviews.llvm.org/D137138



More information about the llvm-commits mailing list