From 82c90724e3b650391f467e8cfeb55934c2ba1dab Mon Sep 17 00:00:00 2001
From: Elliott Hughes <enh@google.com>
Date: Thu, 17 Feb 2022 11:55:49 -0800
Subject: [PATCH] Explain why we test ifunc variables.

Bug: http://b/218788252
Test: treehugger
Change-Id: I51825ab61adeaf6f1242df9a22f89cbd0671db48
---
 tests/dlfcn_test.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/dlfcn_test.cpp b/tests/dlfcn_test.cpp
index 0bf877507..e3664fd97 100644
--- a/tests/dlfcn_test.cpp
+++ b/tests/dlfcn_test.cpp
@@ -258,6 +258,9 @@ TEST(dlfcn, dlopen_vdso) {
   dlclose(handle);
 }
 
+// HWASan uses an ifunc to describe the location of its shadow memory,
+// so even though it's an unusual case, Android needs to support
+// "ifunc variables".
 TEST(dlfcn, ifunc_variable) {
   typedef const char* (*fn_ptr)();