CJ-Johnson added inline comments.
================ Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:1082 +bool isGFX9Plus(const MCSubtargetInfo &STI) { + return isGFX9(STI) || isGFX10(STI); +} ---------------- CJ-Johnson wrote: > arsenm wrote: > > How are these changes related? > This is a mistake. I did not add this and I'm not sure why it is being > included in the change. I believe this has been fixed now. Can you confirm? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D17993/new/ https://reviews.llvm.org/D17993 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits