[Chapel Merge] compiler/next: Stop using public fields for Builde

Notifications Commits
1

Branch: refs/heads/main
Revision: 9744b67
Author: mppf
Log Message:

Merge pull request #18516 from mppf/tidy-builder

compiler/next: Stop using public fields for Builder::Result

  • move Builder::Result to its own file, BuilderResult
  • add accessor methods and make the fields private
  • adjust tests accordingly, including using mod instead of module for
    a variable storing a module, since module is a C++ keyword

There are no functional changes in this PR -- just changing the way the
parsed expressions are accessed.

Reviewed by @dlongnecke-cray - thanks!

  • [x] full local testing....

    Modified Files:
    A compiler/next/include/chpl/uast/BuilderResult.h
    A compiler/next/lib/uast/BuilderResult.cpp
    M compiler/next/include/chpl/parsing/Parser.h
    M compiler/next/include/chpl/parsing/parsing-queries.h
    M compiler/next/include/chpl/uast/Builder.h
    M compiler/next/lib/parsing/Parser.cpp
    M compiler/next/lib/parsing/parsing-queries.cpp
    M compiler/next/lib/uast/Builder.cpp
    M compiler/next/lib/uast/CMakeLists.txt
    M compiler/next/lib/uast/Makefile.include
    M compiler/next/test/parsing/testParse.cpp
    M compiler/next/test/parsing/testParseAggregate.cpp
    M compiler/next/test/parsing/testParseArrayDomainRange.cpp
    M compiler/next/test/parsing/testParseBegin.cpp
    M compiler/next/test/parsing/testParseBracketLoop.cpp
    M compiler/next/test/parsing/testParseCobegin.cpp
    M compiler/next/test/parsing/testParseCoforall.cpp
    M compiler/next/test/parsing/testParseComments.cpp
    M compiler/next/test/parsing/testParseConditional.cpp
    M compiler/next/test/parsing/testParseDefer.cpp
    M compiler/next/test/parsing/testParseDelete.cpp
    M compiler/next/test/parsing/testParseDoWhile.cpp
    M compiler/next/test/parsing/testParseEnums.cpp
    M compiler/next/test/parsing/testParseFor.cpp
    M compiler/next/test/parsing/testParseForall.cpp
    M compiler/next/test/parsing/testParseForeach.cpp
    M compiler/next/test/parsing/testParseFunctions.cpp
    M compiler/next/test/parsing/testParseImport.cpp
    M compiler/next/test/parsing/testParseLabelContinueBreak.cpp
    M compiler/next/test/parsing/testParseLocal.cpp
    M compiler/next/test/parsing/testParseModules.cpp
    M compiler/next/test/parsing/testParseMultiVar.cpp
    M compiler/next/test/parsing/testParseNew.cpp
    M compiler/next/test/parsing/testParseNumericLiterals.cpp
    M compiler/next/test/parsing/testParseOn.cpp
    M compiler/next/test/parsing/testParseReturn.cpp
    M compiler/next/test/parsing/testParseSelect.cpp
    M compiler/next/test/parsing/testParseSerial.cpp
    M compiler/next/test/parsing/testParseStringBytesLiterals.cpp
    M compiler/next/test/parsing/testParseSync.cpp
    M compiler/next/test/parsing/testParseTryCatchThrow.cpp
    M compiler/next/test/parsing/testParseTupleVar.cpp
    M compiler/next/test/parsing/testParseUse.cpp
    M compiler/next/test/parsing/testParseWhile.cpp
    M compiler/next/test/parsing/testParseYield.cpp
    M compiler/next/test/uast/testBuildIDs.cpp
    M compiler/next/test/uast/testVisit.cpp

    Compare: Comparing e18b43c30b0b...9744b67ea12d · chapel-lang/chapel · GitHub