[PATCH] D85225: [Target][AArch64] Allow for char as int8_t in AArch64AsmParser.cpp
Paul Walker via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 18 06:10:41 PDT 2020
paulwalker-arm added inline comments.
================
Comment at: llvm/lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp:859
+ bool IsByte = std::is_same<int8_t, std::make_signed_t<T>>::value ||
+ std::is_same<signed char, std::make_signed_t<T>>::value;
if (auto ShiftedImm = getShiftedVal<8>())
----------------
Perhaps a naive question but are there any concerns about "signed char" being bigger than a byte on any systems?
The commit message suggests its the make_signed that is the problem so I'm wondering if instead adding "std::is_same<int8_t, T>::value" would also work?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85225/new/
https://reviews.llvm.org/D85225
More information about the llvm-commits
mailing list