BeMg added inline comments.
================
Comment at: llvm/unittests/Support/RISCVISAInfoTest.cpp:376
EXPECT_EQ(Exts.count("zihintntl"), 1U);
- auto MaybeISAInfo2 = RISCVISAInfo::parseArchString("rv64izihintntl0p2",
true);
+ auto MaybeISAInfo2 = RISCVISAInfo::parseArchString("rv64izihintntl1p0",
true);
ASSERT_THAT_EXPECTED(MaybeISAInfo2, Succeeded());
----------------
These unit tests are used to test the experimental extension name parsing, it
is not suitable to use zihintntl here because zihintntl is no longer
experimental.
I update it by https://reviews.llvm.org/D155673. After rebasing, I think the
test fail in RISCVISAInfoTest.cpp can be resolved.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151547/new/
https://reviews.llvm.org/D151547
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
