This is an archive of the discontinued LLVM Phabricator instance.

Update LLC tests for slow division bypassing (NFC)
ClosedPublic

Authored by n.bozhenov on Jan 9 2017, 7:19 AM.

Details

Summary

Run update_llc_test_checks.py on

  • CodeGen/X86/atom-bypass-slow-division.ll
  • CodeGen/X86/atom-bypass-slow-division-64.ll
  • CodeGen/X86/slow-div.ll

Diff Detail

Repository
rL LLVM

Event Timeline

n.bozhenov updated this revision to Diff 83621.Jan 9 2017, 7:19 AM
n.bozhenov retitled this revision from to Update LLC tests for slow division bypassing (NFC).
n.bozhenov updated this object.
n.bozhenov added a reviewer: spatel.
n.bozhenov added a subscriber: llvm-commits.
spatel added inline comments.Jan 9 2017, 7:36 AM
test/CodeGen/X86/atom-bypass-slow-division-64.ll
2 ↗(On Diff #83621)

I think some bots will fail unless you specify a triple here. For example, if this runs on a darwin bot, the comment formatting will be different (## instead of #).
My usual triple preference for x86-64 tests is:
x86_64-unknown-unknown

n.bozhenov updated this revision to Diff 83668.Jan 9 2017, 12:35 PM

Ok. I have moved the triple from the bitcode into the RUN line.

n.bozhenov marked an inline comment as done.Jan 9 2017, 12:37 PM
spatel accepted this revision.Jan 9 2017, 3:10 PM
spatel edited edge metadata.

LGTM.

This revision is now accepted and ready to land.Jan 9 2017, 3:10 PM
This revision was automatically updated to reflect the committed changes.