fixup! address review feedback

Author: felipepiovezan

lldb/test/API/macosx/arm-pointer-metadata-cfa-dwarf-expr/TestArmPointerMetadataCFADwarfExpr.py

@@ -15,11 +15,21 @@ def test(self):
         # toggle the bit of fp and save it on the stack:
         # orr   x29, x29, #0x1000000000000000
         # stp	x29, x30, [sp, #-16]!
+        # This is effectively adding metadata to the CFA of the caller frame (main).
         thread.StepInstruction(False)
         thread.StepInstruction(False)
 
+        # The location of `argv` has been artificially made equal to the CFA of the frame.
+        # As such, it should have the metadata artificially set previously.
         argv_addr = thread.frames[1].GetValueForVariablePath("&argv")
         self.assertTrue(argv_addr.IsValid())
-
         argv_addr_uint = argv_addr.GetValueAsUnsigned()
         self.assertNotEqual((argv_addr_uint & (1 << 60)), 0)
+
+        # GetCFA strips metadata.
+        cfa = thread.frames[1].GetCFA()
+        self.assertEqual((cfa & (1 << 60)), 0)
+
+        # If the test worked correctly, the cfa and the location should be identical,
+        # modulo the metadata.
+        self.assertEqual(cfa | (1 << 60), argv_addr_uint)