This is an archive of the discontinued LLVM Phabricator instance.

General abs tests
Needs ReviewPublic

Authored by zoecarver on Apr 5 2019, 5:47 PM.

Details

Summary

Why?

  1. Because there was a lack of abs tests
  2. Because these tests make it very clear what abs declarations are where

Diff Detail

Event Timeline

zoecarver created this revision.Apr 5 2019, 5:47 PM
MaskRay added a subscriber: MaskRay.Apr 5 2019, 6:08 PM
MaskRay added inline comments.
test/std/numerics/numerics.abs/abs_tests.h
11

mixed tabs and spaces?

zoecarver marked an inline comment as done.Apr 5 2019, 7:21 PM
zoecarver added inline comments.
test/std/numerics/numerics.abs/abs_tests.h
11

Arg. The fallout of switching editors (I blame BBEdit). I will fix.