[PATCH] D135614: [OpenMP][CUDA][AMDGPU] Accept case insensitive subarchitecture names

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 10 13:31:45 PDT 2022


tra added a comment.

Is that really something we need/want to do? I've never seen anyone complaining about this particular issue.

clang/gcc are case-sensitive for similar options, like `-march`: https://godbolt.org/z/3jKzTda7h
I think offloading options should behave consistently in that regard. If we do want to make arch/CPU names case-agnostic, it should be done for all such options.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135614



More information about the cfe-commits mailing list