This is an archive of the discontinued LLVM Phabricator instance.

[clang][NFC] Drop alignment in builtin-nondeterministic-value test
ClosedPublic

Authored by ManuelJBrito on Jun 9 2023, 9:10 AM.

Details

Summary

Drop alignment to allow tests to run in different platforms.
Following @zixuan-wu 's request in D142388.

Diff Detail

Event Timeline

ManuelJBrito created this revision.Jun 9 2023, 9:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 9 2023, 9:10 AM
Herald added a subscriber: mgrang. · View Herald Transcript
ManuelJBrito requested review of this revision.Jun 9 2023, 9:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 9 2023, 9:10 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
erichkeane added inline comments.Jun 9 2023, 12:41 PM
clang/test/CodeGen/builtins-nondeterministic-value.c
8–9

this test doesn't seem to be testing for alignment at all, so just take out everything after the type.

Completely drop alignment.

zixuan-wu accepted this revision.Jun 19 2023, 8:14 PM
This revision is now accepted and ready to land.Jun 19 2023, 8:14 PM