[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