[PATCH] D143293: AMDGPU: Use module flag to get code object version at IR level folow-up

[Changpeng Fang via Phabricator via llvm-commits]

cfang added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:807
       break;
-    case ELF::ELFABIVERSION_AMDGPU_HSA_V4:
-    case ELF::ELFABIVERSION_AMDGPU_HSA_V5:
+    case 4:
+    case 5:
----------------
arsenm wrote:
> Going from enum names to magic constants is a regression 
But now we actually switch on CodeObjectVersion which is an unsigned integer, so we have to case on integers like 2, 3, 4
Previous we switch on getHsaAbiVersion, which is an enum value.

In short, I am thinking we can no longer case on enum values now.


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

https://reviews.llvm.org/D143293