This is an archive of the discontinued LLVM Phabricator instance.

Differential D85967

[flang] Add preprocessor test for defines passed on the command line
ClosedPublic

Authored by DavidTruby on Aug 14 2020, 5:26 AM.

Details

Reviewers
sscalpone
klausler
PeteSteinfeld
tskeith
Commits
rG3b338e53e956: [flang] Add preprocessor test for defines passed on the command line
Summary

This adds a test for D85862 to ensure that preprocessor definitions
passed on command lines don't regress in future.

Diff Detail

Event Timeline

DavidTruby created this revision.Aug 14 2020, 5:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 14 2020, 5:26 AM
Herald added a subscriber: llvm-commits. · View Herald Transcript
DavidTruby requested review of this revision.Aug 14 2020, 5:26 AM
Harbormaster completed remote builds in B68402: Diff 285631.Aug 14 2020, 5:34 AM
tskeith accepted this revision.Aug 16 2020, 7:02 AM
tskeith added a subscriber: tskeith.

LGTM

This revision is now accepted and ready to land.Aug 16 2020, 7:02 AM
This revision was landed with ongoing or failed builds.Aug 17 2020, 6:36 AM
Closed by commit rG3b338e53e956: [flang] Add preprocessor test for defines passed on the command line (authored by DavidTruby). · Explain Why
This revision was automatically updated to reflect the committed changes.
DavidTruby added a commit: rG3b338e53e956: [flang] Add preprocessor test for defines passed on the command line.

Revision Contents

PathSize
flang/
test/
Preprocessing/
8 lines

Diff 286003

flang/test/Preprocessing/defines.F90

  • This file was added.
! RUN: %f18 -E -DFOO=1 -DBAR=2 %s | FileCheck %s
! CHECK: integer :: a = 1
integer :: a = FOO
! CHECK: integer :: b = 2
integer :: b = BAR
end program