To allow RAV to visit initializer when bitfield and initializer both given

[NoumanAmir657]

Summary

The RAV does not visit the initializer when the bitfield is given because it uses if-else statement. Code was changed to two seperate ifs so that it can visit both of them if both are present.

[erichkeane]

This needs a test and a release note, and it failed pre-commit CI. I don't have a good idea how to test this however, there is perhaps one of the visitor unit tests that can show this off?

[NoumanAmir657]

The pre-commit checks passed now when I built again. Looking into test for this now

clang/include/clang/AST/RecursiveASTVisitor.h

 DEF_TRAVERSE_DECL(UnnamedGlobalConstantDecl, {})
 
 DEF_TRAVERSE_DECL(TemplateParamObjectDecl, {})
 
 DEF_TRAVERSE_DECL(FieldDecl, {
   TRY_TO(TraverseDeclaratorHelper(D));
   if (D->isBitField())
     TRY_TO(TraverseStmt(D->getBitWidth()));
-  else if (D->hasInClassInitializer())
+  if (D->hasInClassInitializer())
     TRY_TO(TraverseStmt(D->getInClassInitializer()));
 })
 
 DEF_TRAVERSE_DECL(ObjCAtDefsFieldDecl, {
   TRY_TO(TraverseDeclaratorHelper(D));
   if (D->isBitField())
     TRY_TO(TraverseStmt(D->getBitWidth()));
   // FIXME: implement the rest.