This is an archive of the discontinued LLVM Phabricator instance.

test case for BrainF
Needs ReviewPublic

Authored by jnyfah on Apr 29 2021, 4:13 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None

Diff Detail

Event Timeline

jnyfah requested review of this revision.Apr 29 2021, 4:13 PM
jnyfah created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptApr 29 2021, 4:13 PM
xgupta added a comment.May 1 2021, 3:32 AM

Hi @jnyfah!

Pre merge checks are failing which means ninja check-all is not passed. Please see https://llvm.org/docs/Phabricator.html#pre-merge-testing. So before uploading/updating patches, you need to make sure all tests are passing.

And you need to add these tests under llvm/test/examples/ directory instead of llvm/test/. HelloWorld.II file extension should be .ll instead of .LL( you used capital latters).

Any progress on it @jnyfah?

jnyfah added a comment.May 5 2021, 6:32 AM

Yes, still trying to run it with ninja

xgupta added a comment.May 5 2021, 7:57 AM

Fine, feel to ask on forums if you have some questions or getting trouble in understanding concepts.

jnyfah updated this revision to Diff 343912.May 9 2021, 7:46 AM
xgupta removed a reviewer: xgupta.May 14 2021, 1:37 AM
llvm/test/Examples/BrainF/BrainF.ll