search

sstsimulator / sst-macro

SST Macro Element Library
http://sst-simulator.org/
Other
34 stars 41 forks source link

Fix function skip bug for pragmas on FunctionDecls #580

Closed jjwilke closed 3 years ago

jjwilke commented 3 years ago

codes using nullptr pragmas were not building correctly due to the pragmas not being activated. Even though we do not traverse FunctionDecl's without a body, we still need to process pragmas on them.

sst-autotester commented 3 years ago

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request.

sst-autotester commented 3 years ago

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore

  • Build Num: 556
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore

  • Build Num: 840
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore_PY3.8

  • Build Num: 15
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore_PY3.8

  • Build Num: 14
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_nosstcore

  • Build Num: 403
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_withsstcore

  • Build Num: 657
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_nosstcore

  • Build Num: 386
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_withsstcore

  • Build Num: 645
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_Make-Dist

  • Build Num: 128
  • Status: STARTED
## Using Repos:
Repo: CORE (sstsimulator/sst-core)
  • Branch: devel
  • SHA: b484bdb7f999eb15235c89851f2f05426866e146
  • Mode: SUPPORT_REPO
Repo: SQE (sstsimulator/sst-sqe)
  • Branch: devel
  • SHA: 6acbd6dff75843e34dbb88912267378bc9fdc220
  • Mode: SUPPORT_REPO
Repo: ELEMENTS (sstsimulator/sst-elements)
  • Branch: devel
  • SHA: 7d51c969748e1ee5b6b6aeb0cda1dde1600e12c8
  • Mode: SUPPORT_REPO
Repo: MACRO (jjwilke/sst-macro)
  • Branch: fix-fxn-pragma-skip
  • SHA: 430bfd6767bc836ad03be7e8914c2413bf2b25b4
  • Mode: TEST_REPO
Pull Request Author: jjwilke

sst-autotester commented 3 years ago

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore

  • Build Num: 556
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore

  • Build Num: 840
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore_PY3.8

  • Build Num: 15
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore_PY3.8

  • Build Num: 14
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_nosstcore

  • Build Num: 403
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_withsstcore

  • Build Num: 657
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_nosstcore

  • Build Num: 386
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_withsstcore

  • Build Num: 645
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_Make-Dist

  • Build Num: 128
  • Status: PASSED
sst-autotester commented 3 years ago

Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging THE LAST COMMIT TO THIS PULL REQUEST HAS BEEN REVIEWED, BUT NOT ACCEPTED OR REQUIRES CHANGES

sst-autotester commented 3 years ago

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

sst-autotester commented 3 years ago

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

sst-autotester commented 3 years ago

Status Flag 'Pre-Test Inspection' - Auto Inspected - Inspection Is Not Necessary for this Pull Request.

sst-autotester commented 3 years ago

Status Flag 'Pull Request AutoTester' - Testing Jenkins Projects:

Pull Request Auto Testing STARTING (click to expand)

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore

  • Build Num: 557
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore

  • Build Num: 841
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore_PY3.8

  • Build Num: 16
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore_PY3.8

  • Build Num: 15
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_nosstcore

  • Build Num: 404
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_withsstcore

  • Build Num: 658
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_nosstcore

  • Build Num: 387
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_withsstcore

  • Build Num: 646
  • Status: STARTED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_Make-Dist

  • Build Num: 129
  • Status: STARTED
## Using Repos:
Repo: CORE (sstsimulator/sst-core)
  • Branch: devel
  • SHA: b484bdb7f999eb15235c89851f2f05426866e146
  • Mode: SUPPORT_REPO
Repo: SQE (sstsimulator/sst-sqe)
  • Branch: devel
  • SHA: 6acbd6dff75843e34dbb88912267378bc9fdc220
  • Mode: SUPPORT_REPO
Repo: ELEMENTS (sstsimulator/sst-elements)
  • Branch: devel
  • SHA: 7d51c969748e1ee5b6b6aeb0cda1dde1600e12c8
  • Mode: SUPPORT_REPO
Repo: MACRO (jjwilke/sst-macro)
  • Branch: fix-fxn-pragma-skip
  • SHA: adee708cddf50cb5141d5a9b638b23a24fbba024
  • Mode: TEST_REPO
Pull Request Author: jjwilke

sst-autotester commented 3 years ago

Status Flag 'Pull Request AutoTester' - Jenkins Testing: all Jobs PASSED

Pull Request Auto Testing has PASSED (click to expand)

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore

  • Build Num: 557
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore

  • Build Num: 841
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_nosstcore_PY3.8

  • Build Num: 16
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_withsstcore_PY3.8

  • Build Num: 15
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_nosstcore

  • Build Num: 404
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_MacOSX_sst-macro_withsstcore

  • Build Num: 658
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_nosstcore

  • Build Num: 387
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_Ubuntu_sst-macro_withsstcore

  • Build Num: 646
  • Status: PASSED

Build Information

Test Name: SST__AutotestGen2_Devel_Branch_Pull_Request_sst-test_sst-macro_Make-Dist

  • Build Num: 129
  • Status: PASSED
sst-autotester commented 3 years ago

Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging THE LAST COMMIT TO THIS PULL REQUEST HAS NOT BEEN REVIEWED YET!

sst-autotester commented 3 years ago

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

sst-autotester commented 3 years ago

Status Flag 'Pre-Merge Inspection' - - This Pull Request Requires Inspection... The code must be inspected by a member of the Team before Testing/Merging THE LAST COMMIT TO THIS PULL REQUEST HAS BEEN REVIEWED, BUT NOT ACCEPTED OR REQUIRES CHANGES

sst-autotester commented 3 years ago

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

sst-autotester commented 3 years ago

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

sst-autotester commented 3 years ago

All Jobs Finished; status = PASSED, However Inspection must be performed before merge can occur...

jjwilke commented 3 years ago

Yeah, I'll open an issue on being "more idiomatic LLVM"

sst-autotester commented 3 years ago

Status Flag 'Pre-Merge Inspection' - SUCCESS: The last commit to this Pull Request has been INSPECTED AND APPROVED by [ calewis ]!

sst-autotester commented 3 years ago

Status Flag 'Pull Request AutoTester' - Pull Request MUST BE MERGED MANUALLY BY Project Team - This Repo does not support Automerge