Pierre-vh added inline comments.
================ Comment at: llvm/lib/Target/AMDGPU/AsmParser/AMDGPUAsmParser.cpp:5598 return ParseDirectiveHSAMetadata(); } else { - if (IDVal == ".hsa_code_object_version") ---------------- cfang wrote: > Are you sure Non-HSA does not have the four directives you deleted? I'm really not sure, I saw `hsa` in the name and I thought it only applied to HSA, but some tests are failing. I'll leave them in until someone can answer for sure. ================ Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.h:46 enum { - AMDHSA_COV2 = 2, AMDHSA_COV3 = 3, ---------------- cfang wrote: > Should we keep this field, and just mention "unsupported"? I'm not sure about that, I assume that we're removing as many traces of V2 as possible from the backend. No point in keeping an unused enum entry IMO, but I'm okay with keeping it if there's a good reason to ================ Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.h:59 /// false otherwise. bool isHsaAbiVersion3(const MCSubtargetInfo *STI); /// \returns True if HSA OS ABI Version identification is 4, ---------------- cfang wrote: > Are all these "isHsaAbiVersionX" no longer needed? Yes they're needed because they implicitly check for HSA as well Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146023/new/ https://reviews.llvm.org/D146023 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits