[PATCH] D50219: [ADT] Normalize empty triple components
Petr Hosek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 6 13:11:16 PDT 2018
phosek added inline comments.
================
Comment at: llvm/unittests/ADT/TripleTest.cpp:1178
TEST(TripleTest, NormalizeARM) {
- EXPECT_EQ("armv6--netbsd-eabi", Triple::normalize("armv6-netbsd-eabi"));
- EXPECT_EQ("armv7--netbsd-eabi", Triple::normalize("armv7-netbsd-eabi"));
----------------
echristo wrote:
> What should we do here instead?
I'm not sure I understand the question? `armv6-netbsd-eabi` got normalized to `armv6-unknown-netbsd-eabi`, isn't that expected?
Repository:
rL LLVM
https://reviews.llvm.org/D50219
More information about the llvm-commits
mailing list