[PATCH] D135614: [OpenMP][CUDA][AMDGPU] Accept case insensitive subarchitecture names
Artem Belevich via Phabricator via llvm-commits
llvm-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 llvm-commits
mailing list