This is an archive of the discontinued LLVM Phabricator instance.

Move x86-64 builtins from SemaChecking.cpp to BuiltinsX86_64.def
ClosedPublic

Authored by agutowski on Oct 11 2016, 6:35 PM.

Event Timeline

agutowski updated this revision to Diff 74316.Oct 11 2016, 6:35 PM
agutowski retitled this revision from to Move x86-64 builtins from SemaChecking.cpp to BuiltinsX86_64.def.
agutowski updated this object.
agutowski added reviewers: rnk, hans, thakis.
agutowski added a subscriber: cfe-commits.
agutowski added inline comments.Oct 11 2016, 6:37 PM
test/Sema/builtins-x86_64.c
18

Is there some way to ignore all notes? I needed to handle typo-correction notes (like: did you mean '__builtin_ia32_readeflags_u32').

rnk accepted this revision.Oct 12 2016, 8:55 AM
rnk edited edge metadata.

lgtm

test/Sema/builtins-x86_64.c
18

If it's all typo correction, I recommend testing with -fno-spell-checking.

This revision is now accepted and ready to land.Oct 12 2016, 8:55 AM
agutowski updated this revision to Diff 74398.Oct 12 2016, 10:17 AM
agutowski edited edge metadata.

add -fno-spell-checking to tests

agutowski closed this revision.Oct 12 2016, 10:37 AM
lib/Sema/SemaChecking.cpp