This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Error when trying to offload to an unsupported architecture
ClosedPublic

Authored by gtbercea on Apr 13 2017, 11:15 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

gtbercea created this revision.Apr 13 2017, 11:15 AM
gtbercea updated this revision to Diff 95198.Apr 13 2017, 1:46 PM

Re-use an already existing flag rather than creating a new one.

ABataev added inline comments.Apr 14 2017, 6:43 AM
lib/Frontend/CompilerInvocation.cpp
2242–2253

I think you can join these 2 if statements into one.

gtbercea updated this revision to Diff 95461.Apr 17 2017, 11:22 AM

Merge IF statements.

gtbercea marked an inline comment as done.Apr 17 2017, 11:22 AM
This revision is now accepted and ready to land.Apr 17 2017, 11:25 AM
gtbercea closed this revision.Aug 7 2017, 2:11 PM