diff options
| author | Justin Lebar <jlebar@google.com> | 2016-10-28 16:26:26 +0000 |
|---|---|---|
| committer | Justin Lebar <jlebar@google.com> | 2016-10-28 16:26:26 +0000 |
| commit | 2b42ccc78b38e5b11305f7316e108882986f5aa9 (patch) | |
| tree | 382e96370484393998b18a4bf5eba976e84ced42 /clang/lib/Frontend/PrintPreprocessedOutput.cpp | |
| parent | 87a47be039226436fe0562f21f6858a2d244bf17 (diff) | |
| download | bcm5719-llvm-2b42ccc78b38e5b11305f7316e108882986f5aa9.tar.gz bcm5719-llvm-2b42ccc78b38e5b11305f7316e108882986f5aa9.zip | |
[CUDA] [AST] Allow isInlineDefinitionExternallyVisible to be called on functions without bodies.
Summary:
In CUDA compilation, we call isInlineDefinitionExternallyVisible (via
getGVALinkageForFunction) on functions while parsing their definitions.
At the point in time when we call getGVALinkageForFunction, we haven't
yet added the body to the function, so we trip this assert. But as far
as I can tell, this is harmless.
To work around this, we add a new flag to FunctionDecl, "WillHaveBody".
There was other code that was working around the existing assert with a
really awful hack -- this change lets us get rid of that hack.
Reviewers: rsmith, tra
Subscribers: aemerson, cfe-commits
Differential Revision: https://reviews.llvm.org/D25640
llvm-svn: 285410
Diffstat (limited to 'clang/lib/Frontend/PrintPreprocessedOutput.cpp')
0 files changed, 0 insertions, 0 deletions
