This is an archive of the discontinued LLVM Phabricator instance.

[AST] Add fixed-point subtraction constant evaluation.
ClosedPublic

Authored by ebevhan on Jan 22 2020, 6:23 AM.

Diff Detail

Event Timeline

ebevhan created this revision.Jan 22 2020, 6:23 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 22 2020, 6:23 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
rjmccall accepted this revision.Jan 22 2020, 7:12 PM

LGTM other than the request to split the commit.

clang/include/clang/Basic/FixedPoint.h
82

These changes should probably be done in a separate patch.

I don't have a problem with compressing this type, but I'm curious if there was a concrete motivation.

This revision is now accepted and ready to land.Jan 22 2020, 7:12 PM
ebevhan marked an inline comment as done.Jan 23 2020, 12:12 AM
ebevhan added inline comments.
clang/include/clang/Basic/FixedPoint.h
82

I can do it separately.

There wasn't really a strong motivation other than that it could be packed better than it was.

ebevhan updated this revision to Diff 239791.Jan 23 2020, 12:26 AM

Move packing of FixedPointSemantics to a separate patch.

rjmccall accepted this revision.Jan 23 2020, 9:09 AM

LGTM.

ebevhan updated this revision to Diff 255975.Apr 8 2020, 5:09 AM

Rebased.

This revision was automatically updated to reflect the committed changes.