This is an archive of the discontinued LLVM Phabricator instance.

[debuginfo-tests] NFC: Move test that was committed to the wrong location
ClosedPublic

Authored by ldionne on Nov 6 2020, 5:18 AM.

Download Raw Diff

Details

Reviewers

TWeaver
n-omer
jmorse

Commits

rG08e6903eb4df: [debuginfo-tests] NFC: Move test that was committed to the wrong location

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

ldionne created this revision.Nov 6 2020, 5:18 AM

Herald added a project: Restricted Project. · View Herald TranscriptNov 6 2020, 5:18 AM

Herald added subscribers: llvm-commits, jkorous. · View Herald Transcript

ldionne requested review of this revision.Nov 6 2020, 5:18 AM

ldionne mentioned this in D89873: [debuginfo-tests] [dexter] Add test to check debug experience with __attribute__((optnone)).Nov 6 2020, 5:18 AM

ldionne added a project: debug-info.

LGTM. thanks for spotting this... <hangs head in shame>

This revision is now accepted and ready to land.Nov 6 2020, 5:22 AM

Thanks for the catch!

This revision was landed with ongoing or failed builds.Nov 6 2020, 5:24 AM

Closed by commit rG08e6903eb4df: [debuginfo-tests] NFC: Move test that was committed to the wrong location (authored by ldionne). · Explain Why

This revision was automatically updated to reflect the committed changes.

ldionne added a commit: rG08e6903eb4df: [debuginfo-tests] NFC: Move test that was committed to the wrong location.

This is failing on GreenDragon: http://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/24745/

I've temporarily disabled this in 2993850237b089d134cec2776312b61ef9bff757

Revision Contents

Path

Size

	debuginfo-tests/	dexter-tests/
		dexter-tests/

optnone-loops.cpp

dexter-tests/

optnone-loops.cpp

Diff 303412

debuginfo-tests/dexter-tests/optnone-loops.cpp

This file was moved from dexter-tests/optnone-loops.cpp.

The contents of this file were not changed.

dexter-tests/optnone-loops.cpp

This file was moved to debuginfo-tests/dexter-tests/optnone-loops.cpp.