Merge "Revert^2 "Update clang version to clang-r487747"" am: 1784c71446
Original change: https://android-review.googlesource.com/c/platform/development/+/2469787 Change-Id: I5fa2b0752799941aeb0c9fc790ab86e4eeede110 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
@@ -15,5 +15,5 @@
|
|||||||
# limitations under the License.
|
# limitations under the License.
|
||||||
|
|
||||||
export LLVM_BUILD_HOST_TOOLS=true
|
export LLVM_BUILD_HOST_TOOLS=true
|
||||||
export LLVM_PREBUILTS_VERSION=clang-r475365b
|
export LLVM_PREBUILTS_VERSION=clang-r487747
|
||||||
export LLVM_RELEASE_VERSION=16.0.2
|
export LLVM_RELEASE_VERSION=17
|
||||||
|
|||||||
@@ -432,7 +432,7 @@ bool HeaderAbiLinker::ReadExportedSymbolsFromVersionScript() {
|
|||||||
|
|
||||||
repr::VersionScriptParser parser;
|
repr::VersionScriptParser parser;
|
||||||
parser.SetArch(arch_);
|
parser.SetArch(arch_);
|
||||||
parser.SetApiLevel(api_level.getValue());
|
parser.SetApiLevel(api_level.value());
|
||||||
for (auto &&version : excluded_symbol_versions_) {
|
for (auto &&version : excluded_symbol_versions_) {
|
||||||
parser.AddExcludedSymbolVersion(version);
|
parser.AddExcludedSymbolVersion(version);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -103,7 +103,7 @@ VersionScriptParser::ParsedTags VersionScriptParser::ParseSymbolTags(
|
|||||||
} else {
|
} else {
|
||||||
if (!has_introduced_arch_tags) {
|
if (!has_introduced_arch_tags) {
|
||||||
result.has_introduced_tags_ = true;
|
result.has_introduced_tags_ = true;
|
||||||
result.introduced_ = intro.getValue();
|
result.introduced_ = intro.value();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
continue;
|
continue;
|
||||||
@@ -117,7 +117,7 @@ VersionScriptParser::ParsedTags VersionScriptParser::ParseSymbolTags(
|
|||||||
} else {
|
} else {
|
||||||
has_introduced_arch_tags = true;
|
has_introduced_arch_tags = true;
|
||||||
result.has_introduced_tags_ = true;
|
result.has_introduced_tags_ = true;
|
||||||
result.introduced_ = intro.getValue();
|
result.introduced_ = intro.value();
|
||||||
}
|
}
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -25,11 +25,11 @@ TEST(ApiLevelTest, ParseApiLevel) {
|
|||||||
EXPECT_FALSE(ParseApiLevel(""));
|
EXPECT_FALSE(ParseApiLevel(""));
|
||||||
EXPECT_FALSE(ParseApiLevel("A"));
|
EXPECT_FALSE(ParseApiLevel("A"));
|
||||||
|
|
||||||
EXPECT_TRUE(ParseApiLevel("current").hasValue());
|
EXPECT_TRUE(ParseApiLevel("current").has_value());
|
||||||
EXPECT_EQ(FUTURE_API_LEVEL, ParseApiLevel("current").getValue());
|
EXPECT_EQ(FUTURE_API_LEVEL, ParseApiLevel("current").value());
|
||||||
|
|
||||||
EXPECT_TRUE(ParseApiLevel("16").hasValue());
|
EXPECT_TRUE(ParseApiLevel("16").has_value());
|
||||||
EXPECT_EQ(16l, ParseApiLevel("16").getValue());
|
EXPECT_EQ(16l, ParseApiLevel("16").value());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -83,14 +83,14 @@ TEST(StringUtilsTest, ParseInt) {
|
|||||||
EXPECT_FALSE(ParseInt("0xa"));
|
EXPECT_FALSE(ParseInt("0xa"));
|
||||||
EXPECT_FALSE(ParseInt("16h"));
|
EXPECT_FALSE(ParseInt("16h"));
|
||||||
|
|
||||||
EXPECT_TRUE(ParseInt("0").hasValue());
|
EXPECT_TRUE(ParseInt("0").has_value());
|
||||||
EXPECT_EQ(0, ParseInt("0").getValue());
|
EXPECT_EQ(0, ParseInt("0").value());
|
||||||
|
|
||||||
EXPECT_TRUE(ParseInt("16").hasValue());
|
EXPECT_TRUE(ParseInt("16").has_value());
|
||||||
EXPECT_EQ(16, ParseInt("16").getValue());
|
EXPECT_EQ(16, ParseInt("16").value());
|
||||||
|
|
||||||
EXPECT_TRUE(ParseInt("-16").hasValue());
|
EXPECT_TRUE(ParseInt("-16").has_value());
|
||||||
EXPECT_EQ(-16, ParseInt("-16").getValue());
|
EXPECT_EQ(-16, ParseInt("-16").value());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user