This is an archive of the discontinued LLVM Phabricator instance.

[AIX][Clang] Fix XL product name in AIX XL compatibility warning
ClosedPublic

Authored by ZarkoCA on Oct 29 2021, 2:29 PM.

Details

Summary

Correct the XLC/C++ version in the warning message to use the information from
XL's -qversion output.

Diff Detail

Event Timeline

ZarkoCA requested review of this revision.Oct 29 2021, 2:29 PM
ZarkoCA created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptOct 29 2021, 2:29 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
rzurob requested changes to this revision.Nov 1 2021, 7:24 AM

Please change to "IBM XL C/C++ for AIX 16.1.0". i.e. No ", V". Thanks.

This revision now requires changes to proceed.Nov 1 2021, 7:24 AM
ZarkoCA updated this revision to Diff 383804.Nov 1 2021, 7:55 AM
  • Remove V from version number
ZarkoCA updated this revision to Diff 383805.Nov 1 2021, 8:01 AM
  • Remove comma from product name
rzurob accepted this revision.Nov 1 2021, 10:49 AM

LGTM

This revision is now accepted and ready to land.Nov 1 2021, 10:49 AM
This revision was landed with ongoing or failed builds.Nov 5 2021, 10:17 AM
This revision was automatically updated to reflect the committed changes.