[OpenMP] Use the assertion formatting from assert.h
ClosedPublic
Actions

Authored by jhuber6 on Oct 29 2021, 12:51 PM.

Details

Reviewers

jdoerfert
tianshilei1992

Commits

rG2c6a4e5678c2: [OpenMP] Use the assertion formatting from assert.h

Summary

This patch changes the assert_assume function used for internal
assumptions in the device runtime to use a more standard formatting for
the assumption message.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

jhuber6 created this revision.Oct 29 2021, 12:51 PM

Herald added subscribers: guansong, yaxunl. · View Herald TranscriptOct 29 2021, 12:51 PM

jhuber6 requested review of this revision.Oct 29 2021, 12:51 PM

Herald added a project: Restricted Project. · View Herald TranscriptOct 29 2021, 12:51 PM

Herald added subscribers: openmp-commits, sstefan1. · View Herald Transcript

This revision is now accepted and ready to land.Oct 29 2021, 12:56 PM

Harbormaster completed remote builds in B131501: Diff 383469.Oct 29 2021, 1:24 PM

Changing to use assert_fail to avoid duplicatio.

Harbormaster completed remote builds in B131511: Diff 383487.Oct 29 2021, 1:35 PM

Closed by commit rG2c6a4e5678c2: [OpenMP] Use the assertion formatting from assert.h (authored by jhuber6). · Explain WhyOct 29 2021, 1:44 PM

This revision was automatically updated to reflect the committed changes.

jhuber6 added a commit: rG2c6a4e5678c2: [OpenMP] Use the assertion formatting from assert.h.

Revision Contents
Changeset List

Path

Size

openmp/

libomptarget/

DeviceRTL/

include/

Debug.h

12 lines

src/

Debug.cpp

9 lines

Diff 383491

View Options

openmp/libomptarget/DeviceRTL/include/Debug.h