On Wed, Dec 21, 2016 at 9:40 AM, Andreas Boll <andreas.boll....@gmail.com> wrote: > 2016-12-20 16:31 GMT+01:00 Alex Deucher <alexdeuc...@gmail.com>: >> On Tue, Dec 20, 2016 at 6:49 AM, Andreas Boll >> <andreas.boll....@gmail.com> wrote: >>> 2016-12-19 23:45 GMT+01:00 Alex Deucher <alexdeuc...@gmail.com>: >>>> From: Junwei Zhang <jerry.zh...@amd.com> >>>> >>>> v2: use gfxip names for llvm 4.0+ >>>> v3: use tonga for llvm <= 3.8 >>>> >>>> Signed-off-by: Junwei Zhang <jerry.zh...@amd.com> >>>> Reviewed-by: Nicolai Hähnle <nicolai.haeh...@amd.com> >>>> Acked-by: Christian König <christian.koe...@amd.com> >>>> --- >>> >>> snip >>> >>>> diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c >>>> b/src/gallium/drivers/radeon/r600_pipe_common.c >>>> index 0b5c6dc..e0b914c 100644 >>>> --- a/src/gallium/drivers/radeon/r600_pipe_common.c >>>> +++ b/src/gallium/drivers/radeon/r600_pipe_common.c >>>> @@ -755,6 +755,7 @@ static const char* r600_get_chip_name(struct >>>> r600_common_screen *rscreen) >>>> case CHIP_FIJI: return "AMD FIJI"; >>>> case CHIP_POLARIS10: return "AMD POLARIS10"; >>>> case CHIP_POLARIS11: return "AMD POLARIS11"; >>>> + case CHIP_POLARIS12: return "AMD POLARIS12"; >>>> case CHIP_STONEY: return "AMD STONEY"; >>>> default: return "AMD unknown"; >>>> } >>>> @@ -889,9 +890,11 @@ const char *r600_get_llvm_processor_name(enum >>>> radeon_family family) >>>> #if HAVE_LLVM <= 0x0308 >>>> case CHIP_POLARIS10: return "tonga"; >>>> case CHIP_POLARIS11: return "tonga"; >>>> + case CHIP_POLARIS12: return "tonga"; >>>> #else >>>> case CHIP_POLARIS10: return "polaris10"; >>>> case CHIP_POLARIS11: return "polaris11"; >>>> + case CHIP_POLARIS12: return "polaris11"; >>>> #endif >>> >>> You've dropped the processor name for LLVM 4.0+. >>> I guess that wasn't intended. >> >> That was intended. It didn't seem worth adding all of the additional >> special cases. If/when we convert the other asics to use gfxip names, >> we can convert polaris12 as well. >> >> Alex > > Ok, it just wasn't obvious from the updated commit message. Maybe > update the v3 comment before pushing.
Will do. Alex _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev